Repository navigation
GH#2024: ci: force GitHub-hosted runners for fork PRs in addon integration workflow - #2025
Conversation
Completion Summary
aidevops.sh v3.39.49 plugin for OpenCode v1.19.0 with claude-sonnet-5-5 spent 1m and 1,311 tokens on this as a headless worker. Merged via PR #2025 to main. Routing feedback
|
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Fork-aware runner conditions .github/workflows/addon-integration-test.yml |
Both jobs use ubuntu-latest for fork pull requests. Other runs retain the configured runner fallback order. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Bug fix
Possibly related PRs
- Ultimate-Multisite/ultimate-multisite#2022: This PR changes the runner selection introduced in
#2022, while preserving its fallback behavior for other runs.
Merge Risk: 
Minimal · up to f2674
-
The workflow change is mergeable with no actionable current-head risk identified.
Pre-merge checks: 
5
-
Full details: Linked Issues check
-
Explanation
Issue
#2024requires fork pull requests to useubuntu-latestin both jobs while same-repository calls retaininputs.runner,vars.CI_RUNNER, thenubuntu-latest. The diff updates bothphpunitande2ewithgithub.event.pull_request.head.repo.fork && 'ubuntu-latest' || inputs.runner || vars.CI_RUNNER || 'ubuntu-latest'. The changed workflow file matches the issue scope.
Full details: Out of Scope Changes check
-
Explanation
The pull request changes only
.github/workflows/addon-integration-test.yml. Both changes implement the runner restriction required by issue#2024. No unrelated source, test, or configuration changes appear in the reviewed diff.
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. (1 skipped: 1 unsupported.)
-
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.
🔨 Build Complete - Ready for Testing!📦 Download Build Artifact (Recommended)Download the zip build, upload to WordPress and test:
🌐 Test in WordPress Playground (Very Experimental)Click the link below to instantly test this PR in your browser - no installation needed! Login credentials: |
|
Performance Test Results Performance test results for 5a393a1 are in 🛎️! |
Summary
Fork pull requests always use ubuntu-latest in both phpunit and e2e jobs; same-repo callers keep runner input / CI_RUNNER / default. Resolves CodeRabbit finding from #2022.
Files Changed
.github/workflows/addon-integration-test.yml
Runtime Testing
Resolves #2024
aidevops.sh v3.39.49 plugin for OpenCode v1.19.0 with claude-sonnet-5-5 spent 43s and 1,311 tokens on this as a headless worker.
Summary by CodeRabbit