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

fix(evm): Added betanet evm chain id and renamed test_evm to testnet_evm #3800

Merged
merged 2 commits into from
Jan 13, 2021

Conversation

chefsale
Copy link
Contributor

No description provided.

@chefsale chefsale added the T-SRE Team: issues relevant to the SRE team label Jan 12, 2021
@chefsale chefsale added this to the SRE 2021.01.04 - 2021.01.31 milestone Jan 12, 2021
@chefsale chefsale self-assigned this Jan 12, 2021
@artob artob added the A-EVM Area: Native EVM implementation and support label Jan 12, 2021
Copy link
Contributor

@artob artob left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@chefsale
Copy link
Contributor Author

@frol @bowenwang1996 @SkidanovAlex can somebody of the owners take a look at this PR? 😄

@artob
Copy link
Contributor

artob commented Jan 13, 2021

@olonho @willemneal @evgenykuzyakov One more codeowner review needed from you gents...

@artob
Copy link
Contributor

artob commented Jan 13, 2021

@evgenykuzyakov And one last codeowner review from you needed, pretty please.

@evgenykuzyakov
Copy link
Collaborator

Do we need to bump Protocol Version every time?

@chefsale chefsale merged commit daad871 into master Jan 13, 2021
@chefsale chefsale deleted the fix-evm-chain-id-bug branch January 13, 2021 18:34
@artob
Copy link
Contributor

artob commented Jan 13, 2021

Do we need to bump Protocol Version every time?

@evgenykuzyakov In this specific case, no, because I bumped it in #3773 which was merged just prior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-EVM Area: Native EVM implementation and support T-SRE Team: issues relevant to the SRE team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants