Add lint task and fix issues

Taskfile.yml

@@ -8,4 +8,8 @@ tasks:
 
   build:
     cmds:
-      - go build -o ./bin/ ./...
+      - go build -o ./bin/ ./...
+
+  lint:
+    cmds:
+      - golangci-lint run --timeout 5m0s --config .golangci.yml

cmd/ghactions/ghactions.go

@@ -24,8 +24,8 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// GHActionsCmd represents the ghactions command
-func GHActionsCmd() *cobra.Command {
+// CmdGHActions represents the ghactions command
+func CmdGHActions() *cobra.Command {
 	cmd := &cobra.Command{
 		Use:          "ghactions",
 		Short:        "Replace tags in GitHub Actions workflows",
@@ -41,7 +41,7 @@ func GHActionsCmd() *cobra.Command {
 	return cmd
 }
 
-func replace(cmd *cobra.Command, args []string) error {
+func replace(cmd *cobra.Command, _ []string) error {
 	dir := cmd.Flag("dir").Value.String()
 	dryRun, err := cmd.Flags().GetBool("dry-run")
 	if err != nil {

cmd/ghactions/replacer.go

@@ -75,7 +75,9 @@ func (r *replacer) do(ctx context.Context, cmd *cobra.Command) error {
 		return err
 	}
 
-	r.processOutput(cmd, bfs, outfiles)
+	if err := r.processOutput(cmd, bfs, outfiles); err != nil {
+		return err
+	}
 
 	if r.errOnModified && modified {
 		return fmt.Errorf("modified files")

cmd/root.go

@@ -32,7 +32,7 @@ var rootCmd = &cobra.Command{
 
 // Execute runs the root command.
 func Execute() {
-	rootCmd.AddCommand(ghactions.GHActionsCmd())
+	rootCmd.AddCommand(ghactions.CmdGHActions())
 	err := rootCmd.ExecuteContext(context.Background())
 	if err != nil {
 		os.Exit(1)

main.go

@@ -1,6 +1,5 @@
 /*
 Copyright © 2023 NAME HERE <EMAIL ADDRESS>
-
 */
 package main
 

pkg/ghactions/ghactions_test.go

@@ -26,6 +26,8 @@ import (
 )
 
 func TestParseActionReference(t *testing.T) {
+	t.Parallel()
+
 	type args struct {
 		input string
 	}
@@ -88,6 +90,8 @@ func TestParseActionReference(t *testing.T) {
 		tt := tt
 
 		t.Run(tt.name, func(t *testing.T) {
+			t.Parallel()
+
 			gotact, gotref, err := ghactions.ParseActionReference(tt.args.input)
 			if tt.wantErr {
 				require.Error(t, err, "Wanted error, got none")
@@ -101,6 +105,8 @@ func TestParseActionReference(t *testing.T) {
 }
 
 func TestGetChecksum(t *testing.T) {
+	t.Parallel()
+
 	type args struct {
 		action string
 		ref    string
@@ -213,6 +219,8 @@ func TestGetChecksum(t *testing.T) {
 		tt := tt
 
 		t.Run(tt.name, func(t *testing.T) {
+			t.Parallel()
+
 			ghcli := github.NewClient(nil)
 			got, err := ghactions.GetChecksum(context.Background(), ghcli, tt.args.action, tt.args.ref)
 			if tt.wantErr {

pkg/utils/utils.go

@@ -13,6 +13,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+// Package utils provides utilities for boomerang
 package utils
 
 import (