move MaxCodeHashesPerRequest #469
Merged
Conversation
This file contains 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
ceyonur
approved these changes
Jan 25, 2024
marun
approved these changes
Jan 26, 2024
StephenButtolph
approved these changes
Jan 26, 2024
darioush
pushed a commit
that referenced
this pull request
Apr 25, 2024
* fix allow list comments * cleaner sentences * use preallocated big nums * infer allow list role type
darioush
pushed a commit
that referenced
this pull request
Apr 25, 2024
* move inline string ABIs to separate files and embed them (#383) * move inline string ABIs to separate files and embed them * fix tests * fix tests * unexport function * Update accounts/abi/bind/bind.go Co-authored-by: Darioush Jalali <[email protected]> * fix func name Co-authored-by: Darioush Jalali <[email protected]> * replace getByKey with getByAddress (#395) * rework on panics in precompiles (#418) * rework on panics in precompiles * Update precompile/allow_list.go Co-authored-by: aaronbuchwald <[email protected]> * Update precompile/fee_config_manager.go Co-authored-by: aaronbuchwald <[email protected]> * Update precompile/fee_config_manager.go Co-authored-by: aaronbuchwald <[email protected]> * fix reviews * wrap errors in ConfigurePrecompiles * cleaner errors * Update utils.go * Update miner/worker.go Co-authored-by: aaronbuchwald <[email protected]> * Update core/state_processor.go Co-authored-by: aaronbuchwald <[email protected]> Co-authored-by: aaronbuchwald <[email protected]> * Precompile Specific Packages (#420) * rework on panics in precompiles * Update precompile/allow_list.go Co-authored-by: aaronbuchwald <[email protected]> * Update precompile/fee_config_manager.go Co-authored-by: aaronbuchwald <[email protected]> * Update precompile/fee_config_manager.go Co-authored-by: aaronbuchwald <[email protected]> * fix reviews * wrap errors in ConfigurePrecompiles * cleaner errors * move reward manager precompile to package (WIP) * rename files * fix abi path * move typecheck * move precompiles to their own packages * refactor precompile template * remove test file * upate comments * rm test files * new allowlist package * Update precompile/utils.go Co-authored-by: Darioush Jalali <[email protected]> * Update precompile/nativeminter/contract_native_minter.go Co-authored-by: Darioush Jalali <[email protected]> * Update precompile/nativeminter/contract_native_minter.go Co-authored-by: Darioush Jalali <[email protected]> * Update precompile/utils.go Co-authored-by: Darioush Jalali <[email protected]> * Update precompile/nativeminter/contract_native_minter.go Co-authored-by: Darioush Jalali <[email protected]> * fix nits Co-authored-by: aaronbuchwald <[email protected]> Co-authored-by: Darioush Jalali <[email protected]> * rename fee manager config struct (#427) * rename struct * rename fee config managers to fee managers * fix comments * Generalized upgrades rb (#434) * introduce precompiles as registrable modules * add precompile specific contract tests * remove print debug * add unmarshal tests * remove unnecessary func * fix initial disabled value * register all modules in core/evm/contract_stateful * more refactor & test fix * sync template * fix more tests * rename file * add comment * rename * fix linter * use require error contains * remove whitespace * trim mock interface * sort steps * reviews * Update precompile/stateful_precompile_module.go * Update params/precompile_config.go * Update params/precompile_config.go * fix reviews * add new module to configs and group module functions * generalized-upgrades-rb review (#474) * keep genesis disabled fix * nits * nits * nit * review fixes * Update precompile/allowlist/allowlist.go * use address in map * fix linter for embedded keys * update err messages * more err update * remove unnecessary function (#478) * Start work on breaking cyclic dependency (#496) * Update core/state_processor.go * fix reviews * Update precompile/contracts/txallowlist/contract_test.go * Generalized upgrades rb nits0 (#512) * Minor improvements * restore readOnly * more updates * Add back readOnly to allow list tests * Precompile improvements merge (#513) Co-authored-by: aaronbuchwald <[email protected]> Co-authored-by: Ceyhun Onur <[email protected]> Co-authored-by: Darioush Jalali <[email protected]> Co-authored-by: cam-schultz <[email protected]> Co-authored-by: Matthew Lam <[email protected]> Co-authored-by: cam-schultz <[email protected]> Co-authored-by: omahs <[email protected]> Co-authored-by: Anusha <[email protected]> Co-authored-by: Hagen Hübel <[email protected]> Co-authored-by: minghinmatthewlam <[email protected]> Fix: typos (#428) fix allow list comments (#469) fix CGO flags issue (#489) fix lint job (#499) * review fixes * minor nits * fix precompile generator * fix fee manager config test * remove debug files * Update core/state_processor.go Co-authored-by: aaronbuchwald <[email protected]> * fix comments * restore statedb ordering * fix configure in reward manager * precompiles: adds a regression test for the IsDisabled case in AvalancheRules (#515) * Rename configs: alternative (#520) * alternative renaming for precompile configs * fixes * update naming * rename to AllowListConfig * simplify * move blackhole check to module registerer (#523) * move blackhole check to module registerer * check blackhole first * add unit test * Add test case for registering module outside of reserved range --------- Co-authored-by: Aaron Buchwald <[email protected]> * precompile: improve test structure (#517) * refactor precompile tests * minor improvements * nit * fix merge * rename package * pr comments * rm file * merge AllowListTests * pr comments * explicit BeforeHook * wspace * Mark TestTransactionIndices flaky --------- Co-authored-by: Aaron Buchwald <[email protected]> * nit improvements (#529) * nit improvements * move comments to README * Update cmd/precompilegen/template-readme.md * Rename new config (#528) * rename configurator's new config to make config * use new built-in to create new config instance * precompile: just nits (#534)
oxbee
pushed a commit
to taurusgroup/coreth
that referenced
this pull request
Nov 6, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Why this should be merged
Moves the definition of MaxCodeHashesPerRequest from params to
evm.message, to the same place it's defined in subnet-evm.We could move it to
code_request.goinstead in both reposHow this works
Moves a constant
How this was tested
CI