feat: Exclude tag push events from skip-pr-commits logic

When the skip-pr-commits setting is enabled, the system skips push
events for commits that are part of an open pull request to avoid
duplicate pipeline runs. However, this should not apply to tag push
events, as tags represent important release points that should always
trigger pipelines regardless of PR status.

Changes:
- Detect tag push events by checking if ref starts with "refs/tags/"
- Exclude tag push events from skip logic even when
  skip-pr-commits setting is enabled
- Add test case verifying tag push events are not skipped
- Add test case verifying regular push events are still skipped when
  commit is part of a PR

This ensures that tagging a commit (e.g., for releases) always
triggers the associated pipeline runs, while still avoiding duplicate
runs for regular branch commits that are part of pull requests.

Signed-off-by: Zaki Shaikh <[email protected]>

Author: zakisk

docs/content/docs/install/settings.md

@@ -164,6 +164,9 @@ There are a few things you can configure through the ConfigMap
 
   Default: `true`
 
+  **Note:** This setting does not apply to git tag push events. Tag push events will always trigger
+  pipeline runs regardless of whether the tagged commit is part of an open pull request.
+
 {{< support_matrix github_app="true" github_webhook="true" gitea="false" gitlab="false" bitbucket_cloud="false" bitbucket_datacenter="false" >}}
 
 ### Global Cancel In Progress Settings

pkg/provider/github/parse_payload.go

@@ -346,8 +346,10 @@ func (v *Provider) processEvent(ctx context.Context, event *info.Event, eventInt
 			v.Logger.Warnf("Error getting pull requests associated with the commit in this push event: %v", err)
 		}
 
-		// Only check if the flag is enabled and there are pull requests associated with this commit.
-		if v.pacInfo.SkipPushEventForPRCommits && len(prs) > 0 {
+		isGitTagEvent := strings.HasPrefix(gitEvent.GetRef(), "refs/tags/")
+
+		// Only check if the flag is enabled, and there are pull requests associated with this commit, and it's not a tag push event.
+		if v.pacInfo.SkipPushEventForPRCommits && len(prs) > 0 && !isGitTagEvent {
 			isPartOfPR, prNumber := v.isCommitPartOfPullRequest(sha, org, repoName, prs)
 
 			// If the commit is part of a PR, skip processing the push event

pkg/provider/github/parse_payload_test.go

@@ -356,6 +356,18 @@ func TestParsePayLoad(t *testing.T) {
 	samplePRNoRepo.Repo = nil
 	samplePrEventClosed := samplePRevent
 	samplePrEventClosed.Action = github.Ptr("closed")
+
+	sampleGhPRs := []*github.PullRequest{
+		{
+			Number: github.Ptr(41),
+			State:  github.Ptr("closed"),
+		},
+		{
+			Number: github.Ptr(42),
+			State:  github.Ptr("open"),
+		},
+	}
+
 	tests := []struct {
 		name                       string
 		wantErrString              string
@@ -371,6 +383,7 @@ func TestParsePayLoad(t *testing.T) {
 		wantedBranchName           string
 		wantedTagName              string
 		isCancelPipelineRunEnabled bool
+		skipPushEventForPRCommits  bool
 	}{
 		{
 			name:          "bad/unknown event",
@@ -897,6 +910,41 @@ func TestParsePayLoad(t *testing.T) {
 			wantedBranchName:  "main",
 			wantErrString:     "provided SHA samplePRshanew is not the HEAD commit of the branch main",
 		},
+		{
+			name:          "good/skip push event for skip-pr-commits setting",
+			eventType:     "push",
+			triggerTarget: "push",
+			githubClient:  true,
+			payloadEventStruct: github.PushEvent{
+				Ref: github.Ptr("refs/heads/main"),
+				Repo: &github.PushEventRepository{
+					Owner: &github.User{Login: github.Ptr("owner")},
+					Name:  github.Ptr("pushRepo"),
+				},
+				HeadCommit: &github.HeadCommit{ID: github.Ptr("SHAPush")},
+			},
+			shaRet:                    "SHAPush",
+			skipPushEventForPRCommits: true,
+			muxReplies:                map[string]any{"/repos/owner/pushRepo/commits/SHAPush/pulls": sampleGhPRs},
+			wantErrString:             "commit SHAPush is part of pull request #42, skipping push event",
+		},
+		{
+			name:          "good/skip tag push event for skip-pr-commits setting",
+			eventType:     "push",
+			triggerTarget: "push",
+			githubClient:  true,
+			payloadEventStruct: github.PushEvent{
+				Ref: github.Ptr("refs/tags/v1.0.0"),
+				Repo: &github.PushEventRepository{
+					Owner: &github.User{Login: github.Ptr("owner")},
+					Name:  github.Ptr("pushRepo"),
+				},
+				HeadCommit: &github.HeadCommit{ID: github.Ptr("SHAPush")},
+			},
+			shaRet:                    "SHAPush",
+			skipPushEventForPRCommits: true,
+			muxReplies:                map[string]any{"/repos/owner/pushRepo/commits/SHAPush/pulls": sampleGhPRs},
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
@@ -969,7 +1017,7 @@ func TestParsePayLoad(t *testing.T) {
 				ghClient: ghClient,
 				Logger:   logger,
 				pacInfo: &info.PacOpts{
-					Settings: settings.Settings{},
+					Settings: settings.Settings{SkipPushEventForPRCommits: tt.skipPushEventForPRCommits},
 				},
 			}
 			request := &http.Request{Header: map[string][]string{}}