Skip to content

fix(ci): pin build matrix toolchain to 1.95.0#21

Merged
Fahl-Design merged 1 commit into
mainfrom
fix/release-toolchain-pin
May 6, 2026
Merged

fix(ci): pin build matrix toolchain to 1.95.0#21
Fahl-Design merged 1 commit into
mainfrom
fix/release-toolchain-pin

Conversation

@Fahl-Design

Copy link
Copy Markdown
Member

Summary

  • Build matrix in release.yml installed stable toolchain via dtolnay/rust-toolchain, but rust-toolchain.toml pins channel 1.95.0. Targets were added to stable, then cargo activated 1.95.0 without the cross-compilation target → error[E0463]: can't find crate for core on x86_64-apple-darwin.
  • fail-fast cancelled the other archs, so v1.1.0 was published with zero binary assets — install.sh 404s on every platform.
  • Fix: pass toolchain: "1.95.0" and components: clippy, rustfmt to the action so the matrix target is installed against the toolchain cargo actually uses.

Test plan

  • CI Lint & Test job green
  • Once merged: semantic-release cuts v1.1.1
  • All three matrix Build jobs succeed
  • v1.1.1 release contains pvm-linux-x86_64.tar.gz, pvm-macos-aarch64.tar.gz, pvm-macos-x86_64.tar.gz
  • curl -fsSL https://raw.githubusercontent.com/WebProject-xyz/php-version-manager/main/install.sh | bash succeeds on macOS aarch64 (issue Cannot install on MacOs #20 reproducer)

Fixes #20

The build job used dtolnay/rust-toolchain without a toolchain input,
which installs 'stable' and adds the matrix target to it. But
rust-toolchain.toml pins channel 1.95.0, so cargo activates 1.95.0 at
build time without the cross-compilation target installed, producing
'error[E0463]: can't find crate for core' on x86_64-apple-darwin.
fail-fast then cancelled the aarch64 and linux jobs, leaving the v1.1.0
release with zero binary assets and breaking install.sh (404).

Pin the action to toolchain 1.95.0 explicitly and forward components
clippy/rustfmt to match rust-toolchain.toml so the requested target is
installed against the toolchain cargo will actually use.

Fixes #20
@coderabbitai

coderabbitai Bot commented May 6, 2026

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@Fahl-Design has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 47 minutes and 24 seconds before requesting another review.

To continue reviewing without waiting, purchase usage credits in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e3262253-e63c-425f-84d1-ae5f497c2706

📥 Commits

Reviewing files that changed from the base of the PR and between 851380d and 6739862.

📒 Files selected for processing (1)
  • .github/workflows/release.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/release-toolchain-pin

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.

@Fahl-Design
Fahl-Design merged commit 4b8fbdd into main May 6, 2026
3 of 4 checks passed
@Fahl-Design
Fahl-Design deleted the fix/release-toolchain-pin branch May 6, 2026 17:49
@webproject-bot

webproject-bot Bot commented May 6, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 1.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cannot install on MacOs

1 participant