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

eip1559 support #364

Open
wants to merge 11 commits into
base: develop-ready
Choose a base branch
from
Open

eip1559 support #364

wants to merge 11 commits into from

Conversation

HarryLiIsMe
Copy link
Contributor

@HarryLiIsMe HarryLiIsMe commented Jul 5, 2022

  • make sure that you have executed all the following process and no errors occur

    • make fmt
    • make lint
    • make test
  • The major changes of this PR

  1. EIP1559 support
  2. Jenkins tests passed 1: https://jenkins.dev.findora.org/view/integration%20test/job/Manual-Test/99/
  3. Jenkins tests passed 2: https://jenkins.dev.findora.org/view/integration%20test/job/Manual-Test/102/
  • The major impacts of this PR

    • Impact WASM?
    • Impact Web3 API?
    • Impact mainnet data compatibility?
  • Extra documentations

Copy link
Contributor

@simonjiao simonjiao left a comment

Choose a reason for hiding this comment

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

I left some comments and unresolved some previous conversations, please check it if have time.

simonjiao
simonjiao previously approved these changes Jan 19, 2023
@HarryLiIsMe HarryLiIsMe force-pushed the develop_eip1559 branch 2 times, most recently from 1d812ce to cc90645 Compare February 6, 2023 03:57
harryliisme3 and others added 10 commits February 7, 2023 19:20
fix make lint

Eip1559 (#780)

* merge evm runners

* remove legacy action

* use new FeeCalcultor

* fix test evm_tx_hash

* fix gas_price

Co-authored-by: simonjiao <[email protected]>

fix lint
eip1559 wasm support
* Mint FRA for test address

* upgrade prism

* make fmt

* make test

* fix make test

* fix withdraw_fra and parse_deposit_asset
* fix unhandled transaction

* fix evm deliver_tx nonce

* update startnodes.sh and checkpoint file

---------

Co-authored-by: simonjiao <[email protected]>
* honor block_number in call rpc

* add clean_aux()

fmerk update

* enable snapshot

* update storage version

* rename trace/fresh options

* update noah version

* fix build error

---------

Co-authored-by: simonjiao <[email protected]>
Co-authored-by: harry <[email protected]>
@simonjiao simonjiao dismissed their stale review February 8, 2023 05:59

force-pushed code since approved

@tiannian tiannian changed the base branch from develop to develop-ready February 14, 2023 06:11
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.

6 participants