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.

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

Update solidity version requirement #33

Closed
Gooong opened this issue Jun 29, 2022 · 3 comments · Fixed by #35
Closed

Update solidity version requirement #33

Gooong opened this issue Jun 29, 2022 · 3 comments · Fixed by #35
Assignees
@guidiaz

Comments

@Gooong
Copy link

Gooong commented Jun 29, 2022

Hi,

Thanks for your work. This is very helpful. Currently solidity version goes up to 0.8.x. Could we update the solidity version requirement?
@aesedepece
Copy link
Member

Hi @Gooong thanks for the heads up! Will get this done asap.

@aesedepece aesedepece mentioned this issue Jun 30, 2022
Closed
@AHaliq
Copy link

AHaliq commented May 30, 2023

any updates on this?

@guidiaz guidiaz self-assigned this Jun 5, 2023
@guidiaz guidiaz mentioned this issue Jun 6, 2023
Merged
@guidiaz
Copy link
Contributor

guidiaz commented Jun 6, 2023

Some package deps were deleted and the rest upgraded to latest versions. Solidity contracts and test support now any compiler compliant w/ pragma solidity ^0.8.0^, which was the goal of this issue. Unfortunately some test cases are raising arithmetic overflows within the FastEcMul.decomposeScalar(..) method. An assessment and eventual fix will be covered in #36.

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

@guidiaz guidiaz

Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@AHaliq @aesedepece @Gooong @guidiaz