-
Notifications
You must be signed in to change notification settings - Fork 0
Fix CI actions not running in merge queue by adding branches filter #64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: nhorton <204146+nhorton@users.noreply.github.com>
|
@copilot Check the git history - I think we tried targeting main like that. |
You're right - I checked the git history. The previous attempt (in 8592e2b) used My change only added According to the README from that commit, the issue might have been the Should we try |
|
@copilot Yes - try your change with just branches: [main] |
Done! The change with just merge_group:
branches: [main]This is different from the previous attempt that used both |
|
I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request. Posted by the DCO Assistant Lite bot. |
|
@copilot - review the CLA action docs carefully, and maybe open issues, and fix that the CLA Assistant keeps failing when you (Github Copilot Agent) contribute to a PR. |
Co-authored-by: nhorton <204146+nhorton@users.noreply.github.com>
I've reviewed the CLA Assistant configuration and made improvements in commit 7cfcd62: Changes Made:
Current Allowlist: allowlist: 'dependabot[bot],github-actions[bot],claude,copilot,copilot-swe-agent[bot],*[bot]'Key Findings from Documentation:
Potential Issues to Open:
The wildcard pattern should resolve the issue, but if problems continue, an issue should be opened at https://github.com/contributor-assistant/github-action with details about the specific failure. |
Fix CI Actions Not Running in Merge Queue
Solution
Added
branches: [main]filter tomerge_grouptriggers in all CI workflows (withouttypes: [checks_requested]).Changes
branches: [main]to merge_group trigger in validate.ymlbranches: [main]to merge_group trigger in claude-code-test.ymlbranches: [main]to merge_group trigger in cla.ymlConfiguration
All three workflow files now use:
CLA Assistant Fix
Updated the allowlist in
cla.ymlto include*[bot]wildcard pattern, which will automatically allow all bot accounts (includingcopilot-swe-agent[bot]) without requiring explicit additions for each new bot.Allowlist now includes:
dependabot[bot](explicit)github-actions[bot](explicit)claude(explicit)copilot(explicit)copilot-swe-agent[bot](explicit)*[bot](wildcard - catches all bot accounts)Why This Differs from Previous Attempt
The previous fix (commit 8592e2b) tried:
This didn't work and was removed. Our new approach uses only
branches: [main]without thetypesfilter, which may resolve the issue if thetypesfilter was the problematic part.Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.