Skip to content
This repository was archived by the owner on Feb 18, 2025. It is now read-only.

evm: add whitelist deployer v2 #344

Merged
merged 10 commits into from
Sep 26, 2023

Conversation

huyngopt1994
Copy link
Collaborator

@huyngopt1994 huyngopt1994 commented Aug 17, 2023

  • Add whitelist deployer v2
  • Add Unit Test
  • Self Test

@minh-bq
Copy link
Collaborator

minh-bq commented Aug 18, 2023

Please follow the convention in other PRs and squash the commits

@huyngopt1994 huyngopt1994 changed the title Add whitelist deployer v2 evm: add whitelist deployer v2 Aug 18, 2023
@huyngopt1994 huyngopt1994 force-pushed the Add-whitelist-deployer branch from 7d182dc to 00a40c4 Compare August 18, 2023 04:02
@minh-bq
Copy link
Collaborator

minh-bq commented Sep 26, 2023

ronin/params/config.go

Lines 633 to 637 in 83c8375

chainConfigFmt := "{ChainID: %v Homestead: %v DAO: %v DAOSupport: %v EIP150: %v EIP155: %v EIP158: %v Byzantium: %v Constantinople: %v "
chainConfigFmt += "Petersburg: %v Istanbul: %v, Odysseus: %v, Fenix: %v, Muir Glacier: %v, Berlin: %v, London: %v, Arrow Glacier: %v, "
chainConfigFmt += "Engine: %v, Blacklist Contract: %v, Fenix Validator Contract: %v, ConsortiumV2: %v, ConsortiumV2.RoninValidatorSet: %v, "
chainConfigFmt += "ConsortiumV2.SlashIndicator: %v, ConsortiumV2.StakingContract: %v, Puffy: %v, Buba: %v, Olek: %v, Shillin: %v, "
chainConfigFmt += "ConsortiumV2.ProfileContract: %v, ConsortiumV2.FinalityTracking: %v}"

We should input the new configuration here to get it logged on startup

@huyngopt1994 huyngopt1994 merged commit 923653f into axieinfinity:master Sep 26, 2023
andicrypt pushed a commit to andicrypt/ronin that referenced this pull request Nov 1, 2023
* evm: add whitelist deployer v2

* statedb-utils: Add unit testing

* Update Antennablock for testnet

* Add params and update block in genesis

* Update check compatibility and params config
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants