Skip to content

CIP-4127/CIP-4128: Gate Rust CI and declare MSRV - #62

Merged
tobyhede merged 2 commits into
mainfrom
chore/cip-4127-cip-4128-rust-ci
Sep 22, 2026
Merged

tobyhede merged 2 commits into
mainfrom
chore/cip-4127-cip-4128-rust-ci

Conversation

@tobyhede

Copy link
Copy Markdown
Contributor

Summary

  • gate CI on rustfmt, Clippy, and rustdoc warnings
  • fix the six existing Clippy warnings
  • declare Rust 1.94.1 as the MSRV
  • test on both MSRV and stable Rust

Verification

  • cargo fmt --all -- --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • RUSTDOCFLAGS="-D warnings" cargo doc --all-features --no-deps
  • cargo check --no-default-features
  • cargo test on Rust 1.94.1 and stable

@tobyhede
tobyhede requested a review from a team as a code owner September 22, 2026 05:09
Comment thread .github/workflows/test.yml Outdated
name: "🔍 Quality"

steps:
- uses: actions/checkout@v4

@CDThomas CDThomas Sep 22, 2026 •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this is an older version of actions/checkout. Probably worth bumping actions to the latest versions while we're in the area and could also consider pinning to SHAs.

@freshtonic freshtonic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved with suggestion about not using such a recent rust-version in Cargo.toml.

Comment thread Cargo.toml
name = "envelopers"
version = "0.8.3"
edition = "2021"
rust-version = "1.94.1"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The rust-version sets the minimum version of rustc that can successfully compile and run this package.

cargo will refuse to compile the package if rustc does not meet that minimum version requirement. And envelopers is a public package.

We should either set it to the minimum version of rustc that we can successfully compile the package with when --all-features is enabled or pick a version of rustc that is "old enough" to not exclude some TBC portion of rust users in the wild. 1.94.1 was release in March - it's still new-ish.

@tobyhede
tobyhede merged commit 60a9151 into main Sep 22, 2026
4 checks passed
@tobyhede
tobyhede deleted the chore/cip-4127-cip-4128-rust-ci branch September 22, 2026 06:53
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.

3 participants