-
Notifications
You must be signed in to change notification settings - Fork 107
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
Add Zebra crate versions to dev-dependencies, to enable crater tests #6924
Comments
Hey team! Please add your planning poker estimate with Zenhub @arya2 @conradoplg @dconnolly @oxarbitrage @teor2345 @upbqdn |
Hey, I would like to take on that. |
Thank you! It gave me a build error when I tried this, |
indeed it doesn't work for pre-release packages, tho it should work after 1.0.0. Anyway, I'll go with setting specific versions for now |
can you assign this to me please for easier searching? |
Also fyi since #8136 publishing will be blocked by zcash/librustzcash#1083 zcash/librustzcash#1088 |
Motivation
If we add versions to all of Zebra's dev-dependencies, our crates will get tested by Rust compiler team
crater
runs.This helps prevent build and test failures caused by future Rust compiler versions.
Specifications
https://rustc-dev-guide.rust-lang.org/tests/crater.html
Also possibly the restrictions here, if they are still current:
https://www.reddit.com/r/rust/comments/f5xpib/psa_denywarnings_is_actively_harmful/
And we might need optional feature dependencies:
Changes
Testing
This needs testing using
cargo release
, to make sure there aren't any other circular dependencies.Related Work
Split off PR #6908 and ticket #6149.
The text was updated successfully, but these errors were encountered: