fix: print success for linting without errors

jippi · Sep 11, 2024 · a0f82a4 · a0f82a4
1 parent e99d7c3
commit a0f82a4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/cmd/gitlab_lint.go b/cmd/gitlab_lint.go
@@ -67,7 +67,13 @@ func Lint(cCtx *cli.Context) error {
 	}
 
 	// To scm-engine specific linting last
-	return cfg.Lint(ctx, &gitlab.Context{})
+	if err := cfg.Lint(ctx, &gitlab.Context{}); err != nil {
+		return err
+	}
+
+	slogctx.Info(ctx, "No errors found")
+
+	return nil
 }
 
 type EmbedLoader struct{}

diff --git a/pkg/config/action.go b/pkg/config/action.go
@@ -23,7 +23,7 @@ type (
 
 		// (Optional) Only one action per group (in order) will be executed per evaluation cycle.
 		// Use this to 'stop' other actions from running with the same group name
-		Group string `json:"group,omitempty" yaml:"group"`
+		Group string `json:"group,omitempty" yaml:"group,omitempty"`
 
 		// A key controlling if the action should executed or not.
 		//