search

witnet / elliptic-curve-solidity

Elliptic Curve arithmetic operations written in Solidity
MIT License
168 stars 41 forks source link

Update solidity version requirement #33

Closed Gooong closed 1 year ago

Gooong commented 2 years ago

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 commented 2 years ago

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

AHaliq commented 1 year ago

any updates on this?

guidiaz commented 1 year ago

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.