Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update build workflow for CI #2632

Merged
merged 1 commit into from
Jul 22, 2022

Conversation

BlackDex
Copy link
Collaborator

Because we want to support MSRV, we also need to run a CI for this.
This PR adds checks for the MSRV and rust-toolchain defined versions.

It will also run all cargo test, clippy and fmt checks no matter the outcome of the previous job.
This will help when there are multiple issues, like clippy errors and formatting.
Previously it would show only the first failed check and stopped.

It will also output a nice step summary with some details on which checks have failed.
Or it will output a success message.

Because we want to support MSRV, we also need to run a CI for this.
This PR adds checks for the MSRV and rust-toolchain defined versions.

It will also run all cargo test, clippy and fmt checks no matter the outcome of the previous job.
This will help when there are multiple issues, like clippy errors and formatting.
Previously it would show only the first failed check and stopped.

It will also output a nice step summary with some details on which checks have failed.
Or it will output a success message.
@williamdes
Copy link
Contributor

Can you provide a link to what is MSRV?

@BlackDex
Copy link
Collaborator Author

Can you provide a link to what is MSRV?

Sure: https://rust-lang.github.io/rfcs/2495-min-rust-version.html

@dani-garcia dani-garcia merged commit c0efd3d into dani-garcia:main Jul 22, 2022
@BlackDex BlackDex deleted the update-github-actions branch July 22, 2022 14:11
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