fix: don't update pipelines in DRY mode either

jippi · jippi · commit c8059f4255ee · 2024-05-11T22:51:33.000+02:00
diff --git a/pkg/state/context.go b/pkg/state/context.go
@@ -66,7 +66,7 @@ func IsDryRun(ctx context.Context) bool {
 }
 
 func ShouldUpdatePipeline(ctx context.Context) bool {
-	return ctx.Value(updatePipeline).(bool) //nolint:forcetypeassert
+	return !IsDryRun(ctx) && ctx.Value(updatePipeline).(bool) //nolint:forcetypeassert
 }
 
 func MergeRequestID(ctx context.Context) string {

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ func IsDryRun(ctx context.Context) bool {`
`66`	`66`	`}`
`67`	`67`
`68`	`68`	`func ShouldUpdatePipeline(ctx context.Context) bool {`
`69`		`- return ctx.Value(updatePipeline).(bool) //nolint:forcetypeassert`
	`69`	`+ return !IsDryRun(ctx) && ctx.Value(updatePipeline).(bool) //nolint:forcetypeassert`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`func MergeRequestID(ctx context.Context) string {`