CIP-4127/CIP-4128: Gate Rust CI and declare MSRV - #62
Conversation
| name: "🔍 Quality" | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v4 |
There was a problem hiding this comment.
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
left a comment
There was a problem hiding this comment.
Approved with suggestion about not using such a recent rust-version in Cargo.toml.
| name = "envelopers" | ||
| version = "0.8.3" | ||
| edition = "2021" | ||
| rust-version = "1.94.1" |
There was a problem hiding this comment.
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.
Summary
Verification
cargo fmt --all -- --checkcargo clippy --all-targets --all-features -- -D warningsRUSTDOCFLAGS="-D warnings" cargo doc --all-features --no-depscargo check --no-default-featurescargo teston Rust 1.94.1 and stable