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

Update curve25519-dalek to 4.0.0-pre.5 #301

Merged
merged 3 commits into from
Dec 19, 2022

Conversation

daxpedda
Copy link
Contributor

@daxpedda daxpedda commented Dec 16, 2022

Based on facebook/voprf#86 and basically the same.
Again, CI might need to be expanded, happy to add any suggestions to both PRs.

CI will fail until #298.

@daxpedda
Copy link
Contributor Author

Rebased after #298 and facebook/voprf#86.
This is ready to merge, but it's a git dependency until voprf has a new release.

@daxpedda daxpedda marked this pull request as ready for review December 19, 2022 07:10
@daxpedda daxpedda force-pushed the curve25519-update branch 2 times, most recently from a3e8910 to 63159ab Compare December 19, 2022 07:29
@daxpedda
Copy link
Contributor Author

Rebased after #302.

@daxpedda
Copy link
Contributor Author

Removed dependency patch and moved to the new voprf release: facebook/voprf#88.

Copy link
Contributor

@kevinlewi kevinlewi left a comment

Choose a reason for hiding this comment

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

Thanks!!

@kevinlewi kevinlewi merged commit 8901b74 into facebook:main Dec 19, 2022
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.

None yet

3 participants