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

Fixes: https://github.com/woodpecker-ci/woodpecker/issues/3606
This commit is contained in:
Robert Kaussow 2024-04-14 14:25:28 +02:00 committed by GitHub
parent 399bc5bf1b
commit b4cd1da29c
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -42,6 +42,7 @@ func Create(ctx context.Context, _store store.Store, repo *model.Repo, pipeline
return nil, fmt.Errorf(msg) return nil, fmt.Errorf(msg)
} }
if pipeline.Event == model.EventPush || pipeline.Event == model.EventPull || pipeline.Event == model.EventPullClosed {
skipMatch := skipPipelineRegex.FindString(pipeline.Message) skipMatch := skipPipelineRegex.FindString(pipeline.Message)
if len(skipMatch) > 0 { if len(skipMatch) > 0 {
ref := pipeline.Commit ref := pipeline.Commit
@ -51,6 +52,7 @@ func Create(ctx context.Context, _store store.Store, repo *model.Repo, pipeline
log.Debug().Str("repo", repo.FullName).Msgf("ignoring pipeline as skip-ci was found in the commit (%s) message '%s'", ref, pipeline.Message) 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 return nil, ErrFiltered
} }
}
// If the forge has a refresh token, the current access token // If the forge has a refresh token, the current access token
// may be stale. Therefore, we should refresh prior to dispatching // may be stale. Therefore, we should refresh prior to dispatching