Allow webpages to have an empty config (#3417)

bufbuild · Oct 23, 2024 · 81698bb · 81698bb
1 parent 39b1dc4
commit 81698bb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/private/bufpkg/bufcobra/bufcobra.go b/private/bufpkg/bufcobra/bufcobra.go
@@ -77,7 +77,6 @@ func run(
 	if err != nil {
 		return err
 	}
-
 	excludes := slicesext.ToStructMap(config.ExcludeCommands)
 	for _, command := range cobraCommand.Commands() {
 		if _, ok := excludes[command.CommandPath()]; ok {

diff --git a/private/bufpkg/bufcobra/config.go b/private/bufpkg/bufcobra/config.go
@@ -46,8 +46,9 @@ type config struct {
 }
 
 func readConfigFromFile(path string) (*config, error) {
+	var webpagesConfig config
 	if path == "" {
-		return nil, nil
+		return &webpagesConfig, nil
 	}
 	file, err := os.Open(path)
 	if err != nil {
@@ -57,7 +58,6 @@ func readConfigFromFile(path string) (*config, error) {
 	if err != nil {
 		return nil, err
 	}
-	var webpagesConfig config
 	if err := yaml.Unmarshal(data, &webpagesConfig); err != nil {
 		return nil, err
 	}