Skip to content

Comments

Configure workflows to run on merge queue and fix CLA for copilot PRs#47

Merged
nhorton merged 5 commits intomainfrom
copilot/configure-e2e-tests-for-prs
Jan 15, 2026
Merged

Configure workflows to run on merge queue and fix CLA for copilot PRs#47
nhorton merged 5 commits intomainfrom
copilot/configure-e2e-tests-for-prs

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026

The CI workflows needed to be configured to run e2e tests in the merge queue (not on every PR) while maintaining validation on PRs, with concurrency limits and without cron scheduling. Additionally, the CLA workflow needed to be updated to exclude copilot agent authored PRs.

Changes

claude-code-test.yml (e2e tests):

  • Removed pull_request trigger entirely
  • Now runs only on workflow_dispatch (manual) and merge_group (merge queue)
  • Removed schedule cron trigger (was running weekly on Mondays)
  • Simplified claude-code-e2e job condition to check for workflow_dispatch or merge_group events
  • Concurrency controls: cancel-in-progress: true ensures new commits cancel running tests

validate.yml:

  • Added merge_group trigger in addition to existing pull_request trigger
  • Ensures validation (formatting, linting, tests) runs both on PRs and in merge queue

cla.yml:

  • Added copilot-swe-agent[bot] to the CLA allowlist
  • Prevents CLA checks from failing on copilot agent authored PRs

Result

claude-code-test.yml workflow:

  • Runs only in merge queue or via manual dispatch (not on PRs)
  • Reduces API usage by not running e2e tests on every PR commit
  • Validates code before merging to main

validate.yml workflow:

  • Runs on PRs for quick feedback
  • Also runs in merge queue for final verification before merge
  • Cancels in-progress runs when new commits are pushed

cla.yml workflow:

  • Automatically skips CLA signature checks for bot accounts including copilot-swe-agent[bot]
# claude-code-test.yml
on:
  workflow_dispatch:
  merge_group:

# validate.yml  
on:
  pull_request:
  merge_group:

# cla.yml
allowlist: 'dependabot[bot],github-actions[bot],claude,copilot,copilot-swe-agent[bot]'
Original prompt

make it so that the e2e ci tests run on each PR that is ready for review (as opposed to draft). Make sure that there are concurrency limits too so that we kill any running test if the branch updates and a new instance starts. Also make sure that it is running only on the PR and not when the PR merges. Remove any cron triggers it might have


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: nhorton <204146+nhorton@users.noreply.github.com>
Copilot AI changed the title [WIP] Update e2e CI tests to run on ready PRs only Configure e2e tests to run on ready-for-review PRs with concurrency limits Jan 15, 2026
Copilot AI requested a review from nhorton January 15, 2026 23:32
Copy link
Contributor

@nhorton nhorton left a comment

Choose a reason for hiding this comment

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

…ate workflow

Co-authored-by: nhorton <204146+nhorton@users.noreply.github.com>
Copilot AI changed the title Configure e2e tests to run on ready-for-review PRs with concurrency limits Configure workflows to run on merge queue with validation on PRs Jan 15, 2026
Copilot AI requested a review from nhorton January 15, 2026 23:44
@nhorton nhorton marked this pull request as ready for review January 15, 2026 23:47
@github-actions
Copy link
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Developer Certificate of Origin before we can accept your contribution. You can sign the DCO by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


1 out of 2 committers have signed the DCO.
✅ (nhorton)[https://github.com/nhorton]
❌ @Copilot
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the DCO Assistant Lite bot.

@nhorton
Copy link
Contributor

nhorton commented Jan 15, 2026

@copilot - update the CLA github action so that it does not apply to copilot agent authored PRs (like this one that is failing that check).

Co-authored-by: nhorton <204146+nhorton@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 15, 2026

@copilot - update the CLA github action so that it does not apply to copilot agent authored PRs (like this one that is failing that check).

Done in ed3eee3. Added copilot-swe-agent[bot] to the CLA allowlist so the CLA check will skip copilot agent authored PRs.

Copilot AI changed the title Configure workflows to run on merge queue with validation on PRs Configure workflows to run on merge queue and fix CLA for copilot PRs Jan 15, 2026
Copilot AI requested a review from nhorton January 15, 2026 23:52
@nhorton
Copy link
Contributor

nhorton commented Jan 15, 2026

recheck

@nhorton nhorton merged commit 29df4d9 into main Jan 15, 2026
1 of 2 checks passed
@nhorton nhorton deleted the copilot/configure-e2e-tests-for-prs branch January 15, 2026 23:55
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.

2 participants