Skip to content

ci: allow thepastaclaw fork PRs to run on macOS runner#3318

Merged
QuantumExplorer merged 1 commit into
v3.1-devfrom
ci/allow-thepastaclaw-mac-runner
Mar 15, 2026
Merged

ci: allow thepastaclaw fork PRs to run on macOS runner#3318
QuantumExplorer merged 1 commit into
v3.1-devfrom
ci/allow-thepastaclaw-mac-runner

Conversation

@QuantumExplorer

@QuantumExplorer QuantumExplorer commented Mar 15, 2026

Copy link
Copy Markdown
Member

Summary

  • Allow PRs from thepastaclaw's fork to run on the self-hosted macOS ARM64 runner
  • Currently fork PRs skip all Rust tests because the macOS job condition requires head.repo.full_name == github.repository

Context

thepastaclaw has 11 open PRs, none of which can run Rust tests because they're from a fork. The Ubuntu backup runners are also disabled (UBUNTU_BACKUP_ENABLED != true), so zero Rust test coverage runs for any fork PR.

Test plan

  • Once merged, thepastaclaw's PRs will trigger macOS Rust tests on re-push

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated test workflow configuration to improve macOS testing scenarios for contributed pull requests.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions Bot added this to the v3.1.0 milestone Mar 15, 2026
@QuantumExplorer QuantumExplorer merged commit 86a1eee into v3.1-dev Mar 15, 2026
6 of 7 checks passed
@QuantumExplorer QuantumExplorer deleted the ci/allow-thepastaclaw-mac-runner branch March 15, 2026 17:41
@coderabbitai

coderabbitai Bot commented Mar 15, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 786c6ba8-c31d-4c86-b762-7c050f264239

📥 Commits

Reviewing files that changed from the base of the PR and between e2c1039 and abe751f.

📒 Files selected for processing (1)
  • .github/workflows/tests-rs-workspace.yml

📝 Walkthrough

Walkthrough

A single condition is added to the macOS test job guard in the GitHub Actions workflow file, enabling the user 'thepastaclaw' to trigger macOS Rust workspace tests on pull requests from forks.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/tests-rs-workspace.yml
Added conditional logic to macOS test job to permit triggering from fork PRs when the PR owner is 'thepastaclaw'.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A carrot-colored condition takes root,
Letting 'thepastaclaw' forge ahead with boot,
macOS tests now jump at their command,
From forks far and wide across the land! 🌱

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ci/allow-thepastaclaw-mac-runner
📝 Coding Plan
  • Generate coding plan for human review comments

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 and usage tips.

@codecov

codecov Bot commented Mar 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.51%. Comparing base (b0c1667) to head (abe751f).
⚠️ Report is 12 commits behind head on v3.1-dev.

Additional details and impacted files
@@             Coverage Diff              @@
##           v3.1-dev    #3318      +/-   ##
============================================
+ Coverage     71.05%   72.51%   +1.46%     
============================================
  Files          3293     3293              
  Lines        265909   268481    +2572     
============================================
+ Hits         188951   194702    +5751     
+ Misses        76958    73779    -3179     
Components Coverage Δ
dpp 60.13% <ø> (ø)
drive 78.31% <ø> (+0.13%) ⬆️
drive-abci 83.91% <ø> (+0.42%) ⬆️
sdk 31.25% <ø> (ø)
dapi-client 79.06% <ø> (+39.98%) ⬆️
platform-version ∅ <ø> (∅)
platform-value 58.46% <ø> (+18.62%) ⬆️
platform-wallet 60.40% <ø> (ø)
drive-proof-verifier ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant