Skip to content

ci: avoid redundant post-merge runs - #3203

Merged
kevinjqliu merged 2 commits into
mainfrom
kevinjqliu-optimize-merge-queue
Sep 12, 2026
Merged

ci: avoid redundant post-merge runs#3203
kevinjqliu merged 2 commits into
mainfrom
kevinjqliu-optimize-merge-queue

Conversation

@kevinjqliu

@kevinjqliu kevinjqliu commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

What changes are included in this PR?

Remove main push triggers from required workflows. The merge queue already runs the same checks against the final combined change.

Are these changes tested?

Checked with actionlint.

AI Disclosure

Written with GitHub Copilot, reviewed by me.

Merge queue checks already validate the final combined change before it lands, so stop rerunning required workflows on main after merge.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 11, 2026 23:42
@kevinjqliu
kevinjqliu requested review from CTTY and blackmwk September 11, 2026 23:43
@kevinjqliu

Copy link
Copy Markdown
Contributor Author

because we have merge queue now, we dont need to check it again on main branch push 😄

This comment was marked as outdated.

@kevinjqliu
kevinjqliu force-pushed the kevinjqliu-optimize-merge-queue branch 2 times, most recently from 0bbc716 to 859dea4 Compare September 11, 2026 23:49

@CTTY CTTY left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm!

@kevinjqliu
kevinjqliu added this pull request to the merge queue Sep 11, 2026
Merged via the queue into main with commit 29929f6 Sep 12, 2026
23 checks passed
@kevinjqliu
kevinjqliu deleted the kevinjqliu-optimize-merge-queue branch September 12, 2026 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants