Skip to content

ci(test): run the Test workflow on PRs to any base branch#429

Merged
freshtonic merged 1 commit into
mainfrom
ci/test-run-on-all-pr-bases
Jul 24, 2026
Merged

ci(test): run the Test workflow on PRs to any base branch#429
freshtonic merged 1 commit into
mainfrom
ci/test-run-on-all-pr-bases

Conversation

@freshtonic

@freshtonic freshtonic commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

The Test workflow's pull_request trigger had branches: [main], so it only ran for PRs whose base is main. Stacked / queued PRs target intermediate branches (e.g. queue/<name>/...), so CI was skipped on every PR in a stack except the front one.

This drops the branches filter from the pull_request trigger so the workflow runs on pull requests regardless of base branch. The push trigger stays scoped to main (post-merge), and paths-ignore / workflow_dispatch are unchanged.

Separate, standalone change off main (not part of the EQL v3 queue).

Summary by CodeRabbit

  • Tests
    • Continuous integration checks now run for pull requests targeting any branch, including intermediate branches in stacked or queued workflows.

The pull_request trigger had `branches: [main]`, so the workflow only ran
for PRs whose base is main. Stacked / queued PRs target intermediate
branches (e.g. `queue/<name>/...`), so CI was skipped on every PR in a
stack except the front one. Drop the branches filter so it runs on pull
requests regardless of base; the push trigger stays scoped to main.
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: d964c390-aad3-482c-b57a-c62be23bdaee

📥 Commits

Reviewing files that changed from the base of the PR and between 43da7e4 and b54dc04.

📒 Files selected for processing (1)
  • .github/workflows/test.yml

📝 Walkthrough

Walkthrough

The GitHub Actions test workflow now runs for pull_request events targeting any base branch instead of only main, with comments documenting support for stacked and queued pull requests.

Changes

CI trigger coverage

Layer / File(s) Summary
Broaden pull request trigger
.github/workflows/test.yml
Removes the main branch filter from the pull request trigger and documents why non-main target branches require CI execution.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: the Test workflow now runs on pull requests targeting any base branch.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/test-run-on-all-pr-bases

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@freshtonic
freshtonic requested review from auxesis and tobyhede July 24, 2026 03:22
@freshtonic
freshtonic merged commit c78e68f into main Jul 24, 2026
5 checks passed
@freshtonic
freshtonic deleted the ci/test-run-on-all-pr-bases branch July 24, 2026 03:34
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