mirrors/woodpecker
1
0
Fork 0
mirror of https://github.com/woodpecker-ci/woodpecker.git synced 2024-11-22 18:01:02 +00:00
Code Issues Projects Releases Wiki Activity

Ignore gitlab push events without commits (#3339)

Browse source 
Example of the resulting buggy pipeline:


![image](https://github.com/woodpecker-ci/woodpecker/assets/32853499/1e4bedc6-465b-466e-8d89-a7348e374c7f)
This commit is contained in:
Lukas 2024-02-07 01:24:08 +01:00 committed by GitHub
parent db4a50951c
commit baf2e645bc
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
server/forge/gitlab/gitlab.go
View file

@ -655,6 +655,9 @@ func (g *GitLab) Hook(ctx context.Context, req *http.Request) (*model.Repo, *mod
return repo, pipeline, nil
case *gitlab.PushEvent:
if event.TotalCommitsCount == 0 {
return nil, nil, &forge_types.ErrIgnoreEvent{Event: string(eventType), Reason: "no commits"}
}
return convertPushHook(event)
case *gitlab.TagEvent:
return convertTagHook(event)

5
server/forge/types/errors.go
View file

@ -46,11 +46,12 @@ var _ error = new(AuthError)
var ErrNotImplemented = errors.New("not implemented")
type ErrIgnoreEvent struct {
Event string
Event string
Reason string
}
func (err *ErrIgnoreEvent) Error() string {
return fmt.Sprintf("explicit ignored event '%s'", err.Event)
return fmt.Sprintf("explicit ignored event '%s', reason: %s", err.Event, err.Reason)
}
func (*ErrIgnoreEvent) Is(target error) bool {