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

bump gnark-crypto version and match interface changes #1251

Merged

yelhousni merged 1 commit into master from build/bump-gnarkcrypto-vsn

Aug 20, 2024

Contributor

Tabaie commented Aug 16, 2024

This PR updates the Groth16 BSB22 Commitments module to match recent gnark-crypto changes.

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


          bump gnark-crypto version and match interface changes

35f1fce

Tabaie requested a review from yelhousni

August 16, 2024 18:17

Tabaie marked this pull request as ready for review

August 17, 2024 17:44

yelhousni approved these changes

View reviewed changes

Contributor

yelhousni left a comment

LGTM

yelhousni merged commit 841a46e into master

7 checks passed

yelhousni deleted the build/bump-gnarkcrypto-vsn branch

August 20, 2024 20:47

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

Labels

None yet