-
Notifications
You must be signed in to change notification settings - Fork 207
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
rand_core v0.9 support #1642
Comments
Edit: see original version of toplevel description for context
|
I think we can have two sets of methods: one set based on |
|
|
@rozbb perhaps we can get rid of the |
Ya lemme try |
Ok frankly, I'm not sure the tests are really useful. They're not testing anything per se, they're just using the API in a somewhat plausible way. I wouldn't mind if we just deleted them. |
As discussed [here][0] the tests are not particularity useful and block migration to rand_core v0.9. [0]: #1642 (comment)
UPD: The migration is mostly complete. Remaining crates:
kem
:blocked on thehpke
dev dependencyelliptic-curve
: blocked onrand_core
v0.9 support crypto-bigint#759The text was updated successfully, but these errors were encountered: