Summary
The just ci recipe introduced in #7 does not fully match the steps in .github/workflows/ci.yml. This means a PR can pass just ci locally but still fail on CI.
Gaps
| Check |
CI |
just ci |
cargo fmt --all -- --check |
✅ |
❌ missing |
--all-features on check/clippy/test |
✅ |
❌ missing |
cargo doc with RUSTDOCFLAGS="-D warnings" |
✅ |
❌ missing |
Additionally, the .githooks/pre-commit activation comment still references cargo setup (removed in #7).
Changes
- Add
fmt recipe and include it in ci
- Add
--all-features flag to check, lint, and test recipes
- Add
doc recipe with RUSTDOCFLAGS="-D warnings" and include it in ci
- Add standalone
msrv recipe (cargo +1.85 check) — not in ci, requires a second toolchain
- Fix pre-commit hook activation comment:
cargo setup → just setup
Acceptance criteria
Summary
The
just cirecipe introduced in #7 does not fully match the steps in.github/workflows/ci.yml. This means a PR can passjust cilocally but still fail on CI.Gaps
just cicargo fmt --all -- --check--all-featureson check/clippy/testcargo docwithRUSTDOCFLAGS="-D warnings"Additionally, the
.githooks/pre-commitactivation comment still referencescargo setup(removed in #7).Changes
fmtrecipe and include it inci--all-featuresflag tocheck,lint, andtestrecipesdocrecipe withRUSTDOCFLAGS="-D warnings"and include it incimsrvrecipe (cargo +1.85 check) — not inci, requires a second toolchaincargo setup→just setupAcceptance criteria
just cioutput sequence matches thecijob inci.ymljust cipasses locally