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: use a specific Rust nightly version #541

Merged
merged 1 commit into from
May 10, 2022
Merged

Conversation

vmx
Copy link
Contributor

@vmx vmx commented May 9, 2022

Currently Rust nightly is needed as the currently used bellperson version
needs else wouldn't build on ARM64. Using a specific version makes sure
that we don't get arbitrary Clippy failures on CI.

Also fix Clippy warning that comes up with using this nightly version.

@vmx vmx requested a review from raulk May 9, 2022 12:02
@vmx vmx force-pushed the pin-rust-nightly branch 2 times, most recently from 477a16b to 8be8bda Compare May 9, 2022 12:11
Currently Rust nightly is needed as the currently used `bellperson` version
needs else wouldn't build on ARM64. Using a specific version makes sure
that we don't get arbitrary Clippy failures on CI.

Also fix Clippy warning that comes up with using this nightly version.
@vmx
Copy link
Contributor Author

vmx commented May 9, 2022

Sorry for all the force pushing. This should now be on most recent master, with a better fix for the Clippy warning.

@raulk raulk merged commit 26c6f4c into master May 10, 2022
@raulk raulk deleted the pin-rust-nightly branch May 10, 2022 18:50
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