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

feat: remove dalek ng #4769

Merged

Conversation

SWvheerden
Copy link
Collaborator

Description

Removed and replaced the curve25519-dalek-ng dependency and rather use curve25519-dalek as this is the more updated repo.
Updated the version as this is a breaking change with respect to dependencies.
Matched the version of rand to rand_core so that the traits can work.`

How Has This Been Tested?

All unit test pass.

Fixes: #4736

Copy link
Collaborator

@CjS77 CjS77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ut ACK

applications/tari_app_grpc/Cargo.toml Show resolved Hide resolved
@stringhandler stringhandler merged commit 953b0b7 into tari-project:development Oct 5, 2022
@SWvheerden SWvheerden deleted the sw_remove_dalek_ng branch October 5, 2022 10:20
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.

Update dependency to curve25519-dalek from dalek-ng
3 participants