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

Use Rust version 1.60.0 when building for coverage in the CI #689

Conversation

tgonzalezorlandoarm
Copy link
Member

cargo-tarpaulin needs at least Rust 1.60.0 to compile with a version that is newer than two years old. The version is fixed to 0.24.0 because newer versions need at least Rust 1.64.0.

  • To provide coverage with an up-to-date tool, use Rust 1.60.0 for the coverage build in ci.sh.
  • Fix the cargo-tarpaulin version to 0.24.0. This one should be upgraded when the coverage build is upgraded as well.

cargo-tarpaulin needs at least Rust 1.60.0 to compile with a version
that is newer than two years old. The version is fixed to 0.24.0
because newer versions need at least Rust 1.64.0.

 * To provide coverage with an up-to-date tool, use Rust 1.60.0
   for the coverage build in ci.sh.
 * Fix the cargo-tarpaulin version to 0.24.0. This one should be
   upgraded when the coverage build is upgraded as well.

Signed-off-by: Tomás González <[email protected]>
@tgonzalezorlandoarm tgonzalezorlandoarm requested a review from a team as a code owner August 2, 2023 16:54
@tgonzalezorlandoarm
Copy link
Member Author

@ionut-arm @gowthamsk-arm Could I get some input as to whether this is a good idea or whether there is a different approach that I should take?

@tgonzalezorlandoarm
Copy link
Member Author

This would fix all Nightly issues

@tgonzalezorlandoarm tgonzalezorlandoarm changed the title Upgrade coverage build to use Rust version 1.60.0 Use Rust version 1.60.0 when building for coverage in the CI Aug 11, 2023
ci.sh Show resolved Hide resolved
@tgonzalezorlandoarm
Copy link
Member Author

Closing this PR as #692 is opened now.

@tgonzalezorlandoarm tgonzalezorlandoarm deleted the tg/update-ci-msrv branch September 1, 2023 15:58
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