-
Notifications
You must be signed in to change notification settings - Fork 298
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
Version bump pre-release #1536
Version bump pre-release #1536
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fine by me
@@ -3,9 +3,9 @@ members = ["field", "maybe_rayon", "plonky2", "starky", "util"] | |||
resolver = "2" | |||
|
|||
[workspace.dependencies] | |||
ahash = { version = "0.8.3", default-features = false, features = ["compile-time-rng"] } # NOTE: Be sure to keep this version the same as the dependency in `hashbrown`. | |||
ahash = { version = "0.8.7", default-features = false, features = ["compile-time-rng"] } # NOTE: Be sure to keep this version the same as the dependency in `hashbrown`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Be sure to keep this version the same as the dependency in
hashbrown
.
How do you expect to do that? Like literally going to hashbrown
and check which version they're using? Why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When we switched to hashbrown for no-std
, there were some issues with conflicting ahash
, so we resulted in always tracking both together. I don't think that's a big deal, we are not going to be bumping them that often anyway.
Quality Gate passedIssues Measures |
hashbrown
/ahash
pair to get rid of possible compile issues for downstream users (stdsmid
with nightly, for ahash <= 0.8.6, see Bump ahash to fix missingstdsimd
with nightly rust-lang/hashbrown#496)cc @dlubarov @npwardberkeley, as I think you two are the owners on crates.io
closes #1520