Skip to content

GH#2024: ci: force GitHub-hosted runners for fork PRs in addon integration workflow - #2025

Merged
superdav42 merged 1 commit into
mainfrom
feature/auto-20261026-043656-gh2024
Oct 26, 2026
Merged

superdav42 merged 1 commit into
mainfrom
feature/auto-20261026-043656-gh2024

Conversation

@superdav42

@superdav42 superdav42 commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

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

  • Risk level: Low
  • Verification: self-assessed — Workflow YAML expression reviewed; no runtime tests apply (CI workflow change).

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

  • Chores
    • Updated automated checks for pull requests from forks to use a standard hosted runner. Other pull requests continue to use the configured runner when available.
    • No changes to app functionality.

@superdav42 superdav42 added the origin:worker Auto-created by pulse labelless backfill (t2112) label Oct 26, 2026
@superdav42

superdav42 commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator Author

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.
Merged by deterministic merge pass (pulse-wrapper.sh).

Routing feedback

  • Route: standard; 1 attempt, 8 LLM requests, 0 capability escalations, 0 same-tier fallbacks.
  • Population: headless; 0 objective outcomes.
  • Usage: 490,771 tokens; $0.2307 estimated cost.
  • Recommendation: No routing change is recommended from this sample.

@ultimate-multisite ultimate-multisite Bot added the status:in-review Non-draft PR ready for review/merge label Oct 26, 2026
@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 6f62520c-1de2-40a7-bf67-54c3fd53823d
Commits

Reviewing files that changed from the base of the PR and between e3163c7 and f2674e1.

Files selected for processing: 1
  • .github/workflows/addon-integration-test.yml

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

Walkthrough

The workflow now selects ubuntu-latest for fork pull requests in both the phpunit and e2e jobs. Other runs retain the inputs.runner, vars.CI_RUNNER, and ubuntu-latest fallback order.

Changes: Runner Selection

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: Low

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

Change: Bug fix

Possibly related PRs

Merge Risk: Minimal · up to f2674

The workflow change is mergeable with no actionable current-head risk identified.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly identifies the main change: forcing GitHub-hosted runners for fork pull requests in the addon integration workflow.
Linked Issues check Passed Issue #2024 requires fork pull requests to use ubuntu-latest in both jobs while same-repository calls retain inputs.runner, vars.CI_RUNNER, then ubuntu-latest. The diff updates both phpunit …
Out of Scope Changes check Passed 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…
Docstring Coverage Passed 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…
Full details: Linked Issues check

Explanation

Issue #2024 requires fork pull requests to use ubuntu-latest in both jobs while same-repository calls retain inputs.runner, vars.CI_RUNNER, then ubuntu-latest. The diff updates both phpunit and e2e with github.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.

❤️ Share

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

@github-actions

Copy link
Copy Markdown

🔨 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!
Playground support for multisite is very limitied, hopefully it will get better in the future.

🚀 Launch in Playground

Login credentials: admin / password

@github-actions

Copy link
Copy Markdown

Performance Test Results

Performance test results for 5a393a1 are in 🛎️!

@superdav42
superdav42 merged commit 89f3d89 into main Oct 26, 2026
15 checks passed
@superdav42 superdav42 added status:done Task is complete and removed status:in-review Non-draft PR ready for review/merge labels Oct 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

origin:worker Auto-created by pulse labelless backfill (t2112) status:done Task is complete

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review followup: PR #2022 — ci: let addon callers choose the runner via input or CI_RUNNER variable

1 participant