Repository navigation
Run fewer CI jobs on pull requests - #1315
Conversation
The account's 20 hosted runners were full for hours with hundreds of jobs queued. Pull requests now test the oldest and newest Ruby and Rails plus JRuby, with Faraday v1 on the oldest pair: 7 jobs instead of 19. Pushes to main keep every pair, each with Faraday v1. CodeQL moves from the default setup, which runs on every push to main, to a workflow that runs on pull requests and weekly.
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Configure and apply test-matrix selection .github/workflows/_test-matrix.yml, .github/workflows/pr.yml |
The reusable workflow adds a full input that defaults to true. When false, it selects Ruby 3.2, 4.0, and JRuby with Rails 7.1 and 8.1, and runs Faraday v1 only for Ruby 3.2 with Rails 7.1. The pull-request workflow sets full to false. |
CodeQL analysis
| Layer / File(s) | Summary |
|---|---|
Configure and run CodeQL analysis .github/workflows/codeql.yml |
The workflow runs on pull requests targeting main, on a weekly schedule, and on manual dispatch. It defines read-only workflow-level contents permission and concurrency control. Its analysis job uses a language matrix for Actions, JavaScript/TypeScript, and Ruby, then checks out the repository, initializes CodeQL, and analyzes results. |
Priority:
Estimated code review effort: 2 (Simple) | ~12 minutes
Change: Other
Sequence Diagram(s)
sequenceDiagram
participant Runner as GitHub Actions runner
participant Checkout as actions/checkout
participant CodeQL as CodeQL action
Runner->>Checkout: Check out repository
Runner->>CodeQL: Initialize analysis for matrix language
Runner->>CodeQL: Analyze results with language category
Merge Risk: 
4673d
-
The CI changes are mergeable after normal checks; no actionable issue remains.
Pre-merge checks: 
-
Check name Status Explanation Resolution Description check The description clearly explains the matrix reduction, retained full runs on pushes, CodeQL changes, and the reason for the change. However, it omits the required template sections for type of change,… Update the description to include all template headings and complete the applicable checkboxes, including the change type, scope, testing and quality checks, ownership confirmation, and API changes. Mark non-applicable items explicitly. -
Check name Status Explanation Title check The title clearly and concisely describes the main change: reducing CI jobs on pull requests. Docstring Coverage No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0… Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request. Review Evidence Stays Out Of The Repository The pull request changes only three workflow YAML files: .github/workflows/_test-matrix.yml,.github/workflows/codeql.yml, and.github/workflows/pr.yml. It adds no screenshots, captures, compari…Dependencies From Sources The Maintainer Controls The pull request changes only GitHub Actions workflow files. It adds workflow action references, but it does not modify a manifest, lockfile, or package manager configuration. No git URL, fork depende… Full details: Docstring Coverage
-
Explanation
No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (3 skipped: 3 unsupported.)
Full details: Review Evidence Stays Out Of The Repository
-
Explanation
The pull request changes only three workflow YAML files:
.github/workflows/_test-matrix.yml,.github/workflows/codeql.yml, and.github/workflows/pr.yml. It adds no screenshots, captures, comparison pages, recordings, or other review-evidence files.
Full details: Dependencies From Sources The Maintainer Controls
-
Explanation
The pull request changes only GitHub Actions workflow files. It adds workflow action references, but it does not modify a manifest, lockfile, or package manager configuration. No git URL, fork dependency, patch, override, replace entry, or alternate package registry is introduced.
-
Full details: Description check
-
Explanation
The description clearly explains the matrix reduction, retained full runs on pushes, CodeQL changes, and the reason for the change. However, it omits the required template sections for type of change, scope checks, quality checks, ownership, and API changes.
Resolution
Update the description to include all template headings and complete the applicable checkboxes, including the change type, scope, testing and quality checks, ownership confirmation, and API changes. Mark non-applicable items explicitly.
-
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- 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.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
The account's 20 hosted runners were full from 11:00 to 16:30 UTC on 2026-10-26, with up to 380 jobs queued. ruby_llm's PR runs are among the heaviest: 17 test jobs, 16 of which run the whole suite a second time with Faraday v1.
fullinput on_test-matrix.yml(defaulttrue).codeql.yml: pull requests, weekly, and on demand, for the same languages (actions, JavaScript/TypeScript, Ruby). The default setup is already turned off in the repo settings, since GitHub rejects advanced results while it is on.No status checks are required on main, so the smaller PR matrix can't block merges.
Summary by CodeRabbit