Skip to content

Commit

Permalink
PR feedback, small adjustment to manifest AddFile check for site conf…
Browse files Browse the repository at this point in the history
…iguration files. Compare the filename as is since site configuration is looked for only on the project root
  • Loading branch information
marcosnav committed Dec 10, 2024
1 parent 11fadca commit 2b259e7
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions internal/bundles/manifest.go
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ import (
"fmt"
"io"
"sort"
"strings"

"github.com/posit-dev/publisher/internal/clients/connect"
"github.com/posit-dev/publisher/internal/config"
Expand Down Expand Up @@ -206,7 +205,7 @@ func (manifest *Manifest) AddFile(path string, fileMD5 []byte) {

// Update manifest content category if a file being added is a site configuration
for _, ymlFile := range util.KnownSiteYmlConfigFiles {
if strings.HasSuffix(path, ymlFile) {
if path == ymlFile {
manifest.Metadata.ContentCategory = "site"
break
}
Expand Down

0 comments on commit 2b259e7

Please sign in to comment.