Skip to content

add MSRV check to CI - #938

Open
danieleades wants to merge 1 commit into
oxidecomputer:mainfrom
danieleades:msrv
Open

add MSRV check to CI#938
danieleades wants to merge 1 commit into
oxidecomputer:mainfrom
danieleades:msrv

Conversation

@danieleades

@danieleades danieleades commented Dec 20, 2025

Copy link
Copy Markdown
Contributor

Summary

  • declare Rust 1.88 as the workspace MSRV;
  • apply that declaration to all four published packages;
  • add an additive CI job that checks each published package at its declared MSRV;
  • enable Serde's derive feature for the build dependency exposed by --no-dev-deps.

This PR deliberately keeps rust-toolchain.toml, the existing build-and-test matrix and job name, and current action versions unchanged. The contributor-toolchain and broader stable/nightly CI policy is split into #1058.

Validation

cargo hack --workspace --exclude typify-test --exclude example-build --exclude example-macro --no-dev-deps --rust-version check
cargo +1.89.0 fmt --all -- --check
cargo +1.89.0 test --workspace --locked

All commands pass locally; the MSRV checks select Rust 1.88 from the declared package metadata.

@danieleades

Copy link
Copy Markdown
Contributor Author

@ahl is this of interest?

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