Skip to content

ci: simplify msrv check#1673

Merged
Mirko-von-Leipzig merged 2 commits intonextfrom
mirko/ci/msrv
Feb 18, 2026
Merged

ci: simplify msrv check#1673
Mirko-von-Leipzig merged 2 commits intonextfrom
mirko/ci/msrv

Conversation

@Mirko-von-Leipzig
Copy link
Collaborator

@Mirko-von-Leipzig Mirko-von-Leipzig commented Feb 14, 2026

Our msrv check is a nightly run which has been failing recently. Likely due to missing cargo-msrv install requirement if I had to guess (error is opaque).

This PR simplifies the setup by performing the msrv check directly, with each repo package as a separate item in a github matrix. This means we get the direct msrv error output instead of parsing it via a script.

Example run: https://github.com/0xMiden/miden-node/actions/runs/22032125051

@Mirko-von-Leipzig Mirko-von-Leipzig changed the title ci: simplify msrv CI check ci: simplify msrv check Feb 14, 2026
@Mirko-von-Leipzig Mirko-von-Leipzig added no changelog This PR does not require an entry in the `CHANGELOG.md` file CI CI and devops labels Feb 14, 2026
@Mirko-von-Leipzig Mirko-von-Leipzig force-pushed the mirko/ci/msrv branch 11 times, most recently from 999dee1 to 74bd5da Compare February 15, 2026 07:57
@Mirko-von-Leipzig Mirko-von-Leipzig marked this pull request as ready for review February 15, 2026 08:07
@SantiagoPittella
Copy link
Collaborator

Looks good, the linked run is a 404 though. We can replicate this in the client, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI CI and devops no changelog This PR does not require an entry in the `CHANGELOG.md` file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants