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

Bump ed25519-dalek to 2.1.1, ed25519-dalek-bip32 to 0.3.0 #3088

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

kevinheavey
Copy link

@kevinheavey kevinheavey commented Oct 6, 2024

Summary of Changes

  • Bump the two crate versions as in title and fix what breaks
  • This involves one breaking change: Keypair::generate now relies on traits from a newer version of the rand crate - it no longer works with rand 0.7.

This branches off #3087 so that needs to be merged first (update: done).

Thanks to #2752 for figuring out the ed25519-dalek API changes

@kevinheavey kevinheavey force-pushed the ed25519-dalek-2.1.1 branch 3 times, most recently from 20fa549 to e26da48 Compare October 11, 2024 10:16
@kevinheavey kevinheavey force-pushed the ed25519-dalek-2.1.1 branch 2 times, most recently from da64d7e to aecdb9b Compare October 16, 2024 10:51
@kevinheavey kevinheavey force-pushed the ed25519-dalek-2.1.1 branch 3 times, most recently from d2fa89a to 713c148 Compare October 29, 2024 22:16
@kevinheavey kevinheavey force-pushed the ed25519-dalek-2.1.1 branch 2 times, most recently from 06e29a8 to 589a03d Compare November 6, 2024 17:21
@palladionIT
Copy link

Any updates on this? I have a project that would welcome this bump. If there is any way to help out to speed up merging this let me know please.

@kevinheavey
Copy link
Author

The main blocker is that this requires a breaking change

@palladionIT
Copy link

The main blocker is that this requires a breaking change

I am not too deep into the repository, but do you expect a resolution of that anytime soon?

@kevinheavey
Copy link
Author

they are talking about giving these crates their own release schedule but idk when 🤷‍♂️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants