Skip to content

ci: pin the stable rust toolchain - #91

Merged
n13 merged 1 commit into
mainfrom
n13/pin-rust-toolchain
Sep 1, 2026
Merged

ci: pin the stable rust toolchain#91
n13 merged 1 commit into
mainfrom
n13/pin-rust-toolchain

Conversation

@n13

@n13 n13 commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

rust-toolchain pinned channel = "stable", so CI floated to whatever stable was current. Stable 1.98.0 (2026-08-18) added the chunks_exact_to_as_chunks clippy lint, and CI sets RUSTFLAGS: -D warnings, so crates/pow-core/src/lib.rs:15 and :19 became hard errors.

main has been red since then — the 🤖 Analysis (Clippy & Doc) job failed on the merges of #88 and #87, and on every PR opened since, including the automated release PR #89. Nothing in the source changed; only the compiler under it did.

Pin the exact version, the same way chain pins rust-toolchain to 1.93.0 (and its rustfmt-toolchain to a dated nightly). This also puts both repos on one Rust version.

Verified locally on 1.93.0 via the rust-toolchain directory override, with RUSTFLAGS=-D warnings:

  • cargo fmt --all -- --check — clean
  • cargo clippy --locked --workspace --all-features — clean
  • cargo doc --locked --workspace --no-deps --all-features — clean
  • cargo check --locked --workspace --all-targets — clean

No source changes, so once this lands, re-running CI on #89 should go green.

CI resolved the floating stable channel, so Rust 1.98 shipped the
chunks_exact_to_as_chunks clippy lint and failed checks that passed on
an older local stable. Pin the exact version, matching how chain pins
its toolchains.
@n13
n13 merged commit ceff470 into main Sep 1, 2026
5 checks passed
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