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

chore: update Rust version on CI to 1.73 #455

Merged
merged 1 commit into from
May 13, 2024
Merged

chore: update Rust version on CI to 1.73 #455

merged 1 commit into from
May 13, 2024

Conversation

vmx
Copy link
Contributor

@vmx vmx commented May 7, 2024

Use the same rust version on CI as in the rust-toolchain.toml file.

Use the same rust version on CI as in the rust-toolchain.toml file.
Copy link
Member

@rvagg rvagg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we maybe bump the version of both a bit further? ref-fvm is at 1.78 and builtin-actors is at 1.76
looks like rust-fil-proofs is down at 1.70.0 though

@vmx
Copy link
Contributor Author

vmx commented May 13, 2024

should we maybe bump the version of both a bit further?

That's a good question. For me the important part was to have CI in sync with the rust-toolchain.toml. So I\m merging this one.

In case we should have a higher version here, I think it's a separate PR bumping both. Often new Rust versions lead to new Clippy lints, so sometimes also code changes are needed in case the version is bumped.

@vmx vmx merged commit e0d25f3 into master May 13, 2024
9 checks passed
@vmx vmx deleted the ci-rust-173 branch May 13, 2024 11:04
rjan90 added a commit that referenced this pull request May 14, 2024
rvagg pushed a commit that referenced this pull request May 27, 2024
Use the same rust version on CI as in the rust-toolchain.toml file.
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.

2 participants