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.

Sign up for GitHub

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

Jump to bottom

chore: up gnark-crypto #1274

Merged

gbotrel merged 1 commit into master from chore/up-gnark-crypto

Sep 10, 2024

Contributor

yelhousni commented Sep 10, 2024

Description

update gnark-crypto to latest which removes support of BLS12-378 and BW6-756 curves

Type of change

Bug fix (non-breaking change which fixes an issue)

How has this been tested?

wrapped_hash_test.go in std/recursion passes.

How has this been benchmarked?

N/A

Checklist:

I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
I have added tests that prove my fix is effective or that my feature works
I did not modify files generated from templates
golangci-lint does not output errors locally
New and existing unit tests pass locally with my changes
Any dependent changes have been merged and published in downstream modules


          chore: up gnark-crypto

3b4063e

yelhousni requested review from ivokub and gbotrel and removed request for ivokub

September 10, 2024 13:26

yelhousni added bug cleanup labels

ivokub approved these changes

View reviewed changes

Collaborator

ivokub left a comment

LGTM!

gbotrel merged commit e8cb61d into master

6 checks passed

gbotrel deleted the chore/up-gnark-crypto branch

September 10, 2024 13:59

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

Labels