Apply skip ci filter only on push events (#3612)

Fixes: #3606
woodpecker-ci · Apr 14, 2024 · b4cd1da · b4cd1da
1 parent 399bc5b
commit b4cd1da
Showing 1 changed file with 9 additions and 7 deletions.
diff --git a/server/pipeline/create.go b/server/pipeline/create.go
@@ -42,14 +42,16 @@ func Create(ctx context.Context, _store store.Store, repo *model.Repo, pipeline
 		return nil, fmt.Errorf(msg)
 	}
 
-	skipMatch := skipPipelineRegex.FindString(pipeline.Message)
-	if len(skipMatch) > 0 {
-		ref := pipeline.Commit
-		if len(ref) == 0 {
-			ref = pipeline.Ref
+	if pipeline.Event == model.EventPush || pipeline.Event == model.EventPull || pipeline.Event == model.EventPullClosed {
+		skipMatch := skipPipelineRegex.FindString(pipeline.Message)
+		if len(skipMatch) > 0 {
+			ref := pipeline.Commit
+			if len(ref) == 0 {
+				ref = pipeline.Ref
+			}
+			log.Debug().Str("repo", repo.FullName).Msgf("ignoring pipeline as skip-ci was found in the commit (%s) message '%s'", ref, pipeline.Message)
+			return nil, ErrFiltered
 		}
-		log.Debug().Str("repo", repo.FullName).Msgf("ignoring pipeline as skip-ci was found in the commit (%s) message '%s'", ref, pipeline.Message)
-		return nil, ErrFiltered
 	}
 
 	// If the forge has a refresh token, the current access token