fix: only log we change dry-mode run if it differs from server default

jippi · Aug 30, 2024 · e2ded69 · e2ded69
1 parent c2f21dc
commit e2ded69
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/cmd/shared.go b/cmd/shared.go
@@ -113,8 +113,8 @@ func ProcessMR(ctx context.Context, client scm.Client, cfg *config.Config, event
 		return errors.New("cfg==nil; this is unexpected an error, please report!")
 	}
 
-	// Allow controlling 'dry-run' mode via configuration file
-	if cfg.DryRun != nil {
+	// Allow changing the 'dry-run' mode via configuration file
+	if cfg.DryRun != nil && *cfg.DryRun != state.IsDryRun(ctx) {
 		slogctx.Info(ctx, "Configuration file has a 'dry_run' value, using that in favor of server default")
 
 		ctx = state.WithDryRun(ctx, *cfg.DryRun)