Repository navigation
ci: do not cancel target branch check to avoid failed checks on main - #3374
trueberryless merged 1 commit into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update release-source concurrency policy .github/workflows/enforce-release-source.yml, .github/zizmor.yml |
The workflow no longer defines a concurrency group or sets cancel-in-progress. The zizmor configuration excludes this workflow from concurrency-limits and documents the stated concurrency behaviour. |
Suggested reviewers: alexdln, danielroe
Merge Risk: 
Minimal · up to e6330
-
The change removes concurrency restrictions from the release-source check without changing its validation logic. No actionable merge-blocking risk is identified; merge after normal checks pass.
Pre-merge checks: 
4
-
Full details: Title check
-
Explanation
The title names the main change: the workflow no longer cancels the target branch check. The raw summary confirms the concurrency block was removed from enforce-release-source.yml, which matches the title's intent. The title uses the 'ci:' prefix and stays under 70 characters. It does not describe the zizmor ignore rule, but that is a secondary change.
Full details: Description check
-
Explanation
The description states the purpose: fixing the canceled enforce-release-source.yml check on main. It explains removing the concurrency block and adding a zizmor ignore rule. The related-resources links cover earlier attempts, such as setting cancel-in-progress to false and restoring the synchronize trigger. All of this matches the changeset.
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
🧭 Context
Fix red CI on
main.📚 Description
This PR fixes the failing (canceled)
enforce-release-source.ymlcheck, which happens onmainbecause multiple runs are triggered simultaneously there by removing theconcurrencyblock entirely. This also means adding a zizmor ignore rule for this action only, but I think it is the best solution .🔗 Related resources
cancel-in-progresstofalse(by removing it): fb6916bsynchronizetrigger removed for a while: ci: removeeditedtype trigger fromenforce-release-source#2896 (sorry for the confusing PR title), but since the ruleset makes it a necessary status check, the release PR is always blocked after a new commit lands onmain. This is why @danielroe added it again: af01d90