githubaction: Use correct config name for files

Also support old name "bump_files" Was wrong in 7e39445
wader · Sep 16, 2023 · 47b9b0a · 47b9b0a
1 parent 2d05939
commit 47b9b0a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/internal/github/action.go b/internal/github/action.go
@@ -65,6 +65,9 @@ func NewActionEnv(getenv GetenvFn, version string) (*ActionEnv, error) {
 		return nil, err
 	}
 	repositoryParts := strings.SplitN(repository, "/", 2)
+	if len(repositoryParts) < 2 {
+		return nil, fmt.Errorf("GITHUB_REPOSITORY has invalid value %q", repository)
+	}
 
 	client := &Client{
 		Token:   token,

diff --git a/internal/githubaction/githubaction.go b/internal/githubaction/githubaction.go
@@ -118,7 +118,9 @@ func (c Command) run() []error {
 		return []error{fmt.Errorf("GITHUB_SHA not set")}
 	}
 
-	files, _ := ae.Input("bump_files")
+	// support "bump_files" for backward compatibility
+	bumpFiles, _ := ae.Input("bump_files")
+	files, _ := ae.Input("files")
 	var bumpfile,
 		titleTemplate,
 		commitBodyTemplate,
@@ -159,8 +161,10 @@ func (c Command) run() []error {
 	}
 
 	// TODO: whitespace in filenames
-	filesParts := strings.Fields(files)
-	bfs, errs := bump.NewBumpFileSet(c.OS, all.Filters(), bumpfile, filesParts)
+	var filenames []string
+	filenames = append(filenames, strings.Fields(bumpFiles)...)
+	filenames = append(filenames, strings.Fields(files)...)
+	bfs, errs := bump.NewBumpFileSet(c.OS, all.Filters(), bumpfile, filenames)
 	if errs != nil {
 		return errs
 	}
@@ -185,7 +189,7 @@ func (c Command) run() []error {
 			continue
 		}
 
-		fmt.Printf("  Updateable to %s\n", check.Latest)
+		fmt.Printf("  Updatable to %s\n", check.Latest)
 
 		templateReplacerFn := CheckTemplateReplaceFn(check)