We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
iden3/snarkjs#464 was merged and is now included in snarkjs 0.7.5 so we can remove the special case for plonk verifiers:
https://github.com/erhant/circomkit/blob/main/tests/circomkit.test.ts#L97
The way snarkjs is set as a peerDependency for 0.7.x, it would need to be updated to ^0.7.5 too I would imagine though
0.7.x
^0.7.5
The text was updated successfully, but these errors were encountered:
good point!
The way snarkjs is set as a peerDependency for 0.7.x, it would need to be updated to ^0.7.5 too I would imagine though indeed
will include this for some minor version update soon, but lets wait a bit until SnarkJS gets updated around the place 👍🏻
Sorry, something went wrong.
the existing fix does not break with the fixed snarksj version though right?
numtel
No branches or pull requests
iden3/snarkjs#464 was merged and is now included in snarkjs 0.7.5 so we can remove the special case for plonk verifiers:
https://github.com/erhant/circomkit/blob/main/tests/circomkit.test.ts#L97
The way snarkjs is set as a peerDependency for
0.7.x
, it would need to be updated to^0.7.5
too I would imagine thoughThe text was updated successfully, but these errors were encountered: