Skip to content

Conversation

@jules
Copy link

@jules jules commented Feb 9, 2022

Fixes #21

@jules jules self-assigned this Feb 9, 2022
Copy link

@0xmountaintop 0xmountaintop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@0xmountaintop
Copy link

BTW, maybe also a good idea to add comments in the codes to explain why we would like to keep the ethereum's version

@0xmountaintop
Copy link

0xmountaintop commented Feb 9, 2022

after another look I think we should use go-ethereum v1.10.13, to be consistent with the statement in our readme

go.sum Outdated
github.com/envoyproxy/go-control-plane v0.9.1-0.20191026205805-5f8ba28d4473/go.mod h1:YTl/9mNaCwkRvm6d1a2C3ymFceY/DCBVvsKhRF0iEA4=
github.com/envoyproxy/protoc-gen-validate v0.1.0/go.mod h1:iSmxcyjqTsJpI2R4NaDN7+kN2VEUnK/pcBlmesArF7c=
github.com/ethereum/go-ethereum v1.10.15 h1:E9o0kMbD8HXhp7g6UwIwntY05WTDheCGziMhegcBsQw=
github.com/ethereum/go-ethereum v1.10.15/go.mod h1:W3yfrFyL9C1pHcwY5hmRHVDaorTiQxhYBkKyu5mEDHw=
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We'd better stay the same version v1.10.13 !

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just changed it, should be on 1.10.13 now :)

@0xmountaintop 0xmountaintop merged commit 59df2d7 into scroll-tech:zkrollup Feb 9, 2022
0xmountaintop added a commit that referenced this pull request Apr 19, 2023
0xmountaintop added a commit that referenced this pull request Apr 19, 2023
* Revert "Substitute secp256k1 library for the L1 version (#22)"

This reverts commit 59df2d7.

* goimports -local github.com/scroll-tech/go-ethereum -w .

* go mod tidy
pdrobnjak pushed a commit to Tenderly/net-scroll-geth that referenced this pull request Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove secp256k1 package

3 participants