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.

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

ETC Atlantis fork #179

Merged
merged 18 commits into from Nov 15, 2019

Conversation

@edwardmack
Copy link
Contributor

edwardmack commented Nov 9, 2019

PR description

This PR introduces code to support ECIP 1054: Atlantis EVM and Protocol Upgrades

Fixed Issue(s)

N/a

edwardmack added 7 commits Nov 8, 2019
This includes code to address reading config for classic network and
kotti test network.

ClassicForkPeerValidator checks peers at classicForkBlock (same as Dao
Fork Block) and connects with peers that did not do the Dao Fork and
continued on the classic network.

Signed-off-by: edwardmack <ed@edwardmack.com>
Code to support ECIP-1015 gas calculator and ECIP-1010 difficulty
calculator.

Signed-off-by: edwardmack <ed@edwardmack.com>
Code for Gotham Classic fork supporting ECIP-1017, ECIP-1039 and
ECIP-1041.

Signed-off-by: edwardmack <ed@edwardmack.com>
Signed-off-by: edwardmack <ed@edwardmack.com>
Signed-off-by: edwardmack <ed@edwardmack.com>
Code to support ECIP 1054: Atlantis EVM and Protocol Upgrades

Signed-off-by: edwardmack <ed@edwardmack.com>
@lgtm-com

This comment has been minimized.

Copy link

lgtm-com bot commented Nov 9, 2019

This pull request introduces 1 alert when merging 9e895c9 into 7ec9530 - view on LGTM.com

new alerts:

  • 1 for Expression always evaluates to the same value
edwardmack added 2 commits Nov 11, 2019
Use SpuriousDragonGasCalculator for Atlantis fork due to difference in
the way that new account gas cost is calculated.

Signed-off-by: edwardmack <ed@edwardmack.com>
@lgtm-com

This comment has been minimized.

Copy link

lgtm-com bot commented Nov 11, 2019

This pull request introduces 1 alert when merging f98f55c into 0a27885 - view on LGTM.com

new alerts:

  • 1 for Expression always evaluates to the same value
edwardmack added 8 commits Nov 12, 2019
Signed-off-by: edwardmack <ed@edwardmack.com>
Signed-off-by: edwardmack <ed@edwardmack.com>
Signed-off-by: edwardmack <ed@edwardmack.com>
Signed-off-by: edwardmack <ed@edwardmack.com>
@shemnon

This comment has been minimized.

Copy link
Member

shemnon commented Nov 15, 2019

References tests again got reverted.

cd ethereum/referencetests/src/test/resources
git checkout v7.0.0-beta.1
cd ../../../../..
git add ethereum/referencetests/src/test/resources
git commit --signoff -m "update reference tests"
@shemnon shemnon merged commit 9b9c373 into hyperledger:master Nov 15, 2019
10 checks passed
10 checks passed
DCO DCO
Details
LGTM analysis: Java No new or fixed alerts
Details
LGTM analysis: JavaScript No new or fixed alerts
Details
ci/circleci: acceptanceTests Your tests passed on CircleCI!
Details
ci/circleci: assemble Your tests passed on CircleCI!
Details
ci/circleci: buildDocker Your tests passed on CircleCI!
Details
ci/circleci: integrationTests Your tests passed on CircleCI!
Details
ci/circleci: referenceTests Your tests passed on CircleCI!
Details
ci/circleci: unitTests Your tests passed on CircleCI!
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details
@edwardmack edwardmack deleted the ChainSafe:ed/etc_atlantis_fork branch Nov 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.