Skip to content

chore: pin Rust toolchain to 1.96.0 - #21

Merged
apriendeau merged 2 commits into
mainfrom
austin/upgrade-to-rust-1.96
Jun 3, 2026
Merged

apriendeau merged 2 commits into
mainfrom
austin/upgrade-to-rust-1.96

Conversation

@apriendeau

Copy link
Copy Markdown
Contributor

Summary

Pins the project to Rust 1.96.0 by adding a rust-toolchain.toml:

[toolchain]
channel = "1.96.0"
components = ["clippy", "rustfmt"]
profile = "minimal"

Previously nothing pinned a version — no rust-toolchain.toml, no MSRV in Cargo.toml, and CI just installed stable. This ensures all developers and tooling use a consistent toolchain.

Verification

  • rustup show confirms 1.96.0 is overridden by the toolchain file, with clippy + rustfmt present.
  • just check passes clean: fmt, clippy (-D warnings), build, all tests, and cargo check --locked.

Out of scope (intentionally)

  • CI still uses stable — the workflows run rustup default stable, which overrides the toolchain file. CI won't actually use 1.96.0 until those lines change.
  • No MSRV (rust-version) declared in Cargo.toml.

⚠️ Note: the CI version-check job requires Cargo.toml version to be bumped from main. This PR doesn't change the version, so that check will fail until the version is bumped (or the check is waived).

🤖 Generated with Claude Code

Add rust-toolchain.toml pinning channel 1.96.0 with clippy and rustfmt
components so all developers and tooling use a consistent toolchain.
Previously nothing pinned a version (CI used stable, no MSRV declared).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@apriendeau apriendeau self-assigned this Jun 3, 2026
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@apriendeau
apriendeau merged commit 5394949 into main Jun 3, 2026
3 checks passed
@apriendeau
apriendeau deleted the austin/upgrade-to-rust-1.96 branch June 3, 2026 20:39
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