EIP-3541 ethereum/tests Run#1246
Closed
holgerd77 wants to merge 58 commits into
Closed
Conversation
tx: create access list utility
tx: export EIP1559
common: add docs getEIPActivationBlockNumber common: add extra constant
block: add more EIP1559 tests
block: enable access list transactions as TxData and EIP1559 transactions block: add base fee test check
tx: add EIP1559-related gas logic block: use EIP1559 api from tx
tx: fix gas limit not present in signed EIP1559 txns tx/vm: lint
…rice (with extended 0x-instantiation guard) to legacy and EIP-2930 txs
…igned default value instantiation for fromValuesArray() calls
…uctor, introduced TRANSACTION_TYPE and TRANSACTION_TYPE_BUFFER constants to EIP-2930 tx, added v, r, s aliases for consistency
…two fixes on TRANSACTION_TYPE_BUFFER Buffer initialization, fixed transaction type Buffer comparison in fromSerializedTx()
Co-authored-by: Ryan Ghods <ryan@ryanio.com>
vm: fix tests
…le, added EIP-2930 as required EIP
…for FeeMarketEIP1559TxData interface
…mTxData() instantiation, added never type for FeeMarketEIP1559TxData gasPrice, deprecated TransactionFactory.getTransactionClass() method
…n, moved optional type property to TxData type
…e() API, added code docs
…too-much-gas-used check out of EIP-1559 section
…ackwards compatibility
Member
Author
|
Closed in favor of #1247 (wrong base branch) |
This was referenced Aug 20, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is a test run of a our EIP-3541 implementation from #1240 towards the tests implemented at ethereum/tests#835.
DO NOT MERGE!!!
@axic I noted that our test setup is very much not prepared to adhoc integrate these invented fork names, I couldn't imminently get this work to run on CI.
I will instead take this PR as a base and do manual tests runs of the different files.