8000 Enable SCA overwriting over EOA after Shanghai hardfork by aidan-kwon · Pull Request #1904 · klaytn/klaytn · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Aug 19, 2024. It is now read-only.

Enable SCA overwriting over EOA after Shanghai hardfork #1904

Merged
merged 3 commits into from
Jul 27, 2023

Conversation

aidan-kwon
Copy link
Member

Proposed changes

  • Since d5f24d5 changes the semantics of EVM, Klaytn includes the change in the ShanghaiCompatible hardfork item list.
  • Enable "Shanghai" hardfork keyword in klaytn-test test cases

NOTE: This PR should be merged after merging klaytn/klaytn-tests#10 to execute CI tests successfully.

Types of changes

Please put an x in the boxes related to your change.

  • Bugfix
  • New feature or enhancement
  • Others

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

  • I have read the CONTRIBUTING GUIDELINES doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes ($ make test)
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules

@aidan-kwon aidan-kwon self-assigned this Jul 26, 2023
@aidan-kwon aidan-kwon added the need to merge Need to merge for the next time label Jul 26, 2023
@aidan-kwon
Copy link
Member Author

Reviewers, PTAL

< 8000 /div>
@aidan-kwon aidan-kwon merged commit e823285 into klaytn:dev Jul 27, 2023
@aidan-kwon aidan-kwon deleted the 230726-shanghai-SCA branch July 27, 2023 07:46
@aidan-kwon aidan-kwon mentioned this pull request Aug 3, 2023
20 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
need to merge Need to merge for the next time
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0