Skip to content

Comments

feat: new gas params, tx initial gas and codedeposit#3260

Merged
rakita merged 34 commits intomainfrom
rakita/gas-params-codedeposit
Jan 8, 2026
Merged

feat: new gas params, tx initial gas and codedeposit#3260
rakita merged 34 commits intomainfrom
rakita/gas-params-codedeposit

Conversation

@rakita
Copy link
Member

@rakita rakita commented Dec 29, 2025

Add codedeposit to GasParams and other missing gas params values.

@codspeed-hq
Copy link

codspeed-hq bot commented Dec 29, 2025

CodSpeed Performance Report

Merging this PR will not alter performance

Comparing rakita/gas-params-codedeposit (deb1680) with main (7f3083b)

Summary

✅ 173 untouched benchmarks

@rakita rakita changed the title Rakita/gas params codedeposit feat: new gas params, tx initial gas and codedeposit Jan 5, 2026
Base automatically changed from rakita/cfg-gas-params to main January 8, 2026 10:38
@rakita rakita merged commit 281c6ad into main Jan 8, 2026
2 checks passed
@github-actions github-actions bot mentioned this pull request Jan 8, 2026
@rakita rakita deleted the rakita/gas-params-codedeposit branch January 8, 2026 11:03
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 21, 2026
…3260)

* feat: move GasParams to Cfg

* propagate spec id

* use cfg spec when evm is created

* nits

* nit

* doc tests

* use derive-where to skip fn

* Move SetSpecTr and use it as main tr

* feat: add codedeposit price in GasParams

* remove SetSpecTr, introduce is_custom_gas_param flag

* simplification

* nits and tests fix

* morf spec type

* dont compare ptr

* import fix

* typo

* ref alloy-eips

* add core::error::Error impl to BalError

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>

* send/sync on GasParams

* relex generic for some Cfg functions

* load_account_mut_skip_cold

* WIP

* move initial tx gas calc to GasParams

* fixes

---------

Co-authored-by: Claude Opus 4.5 <[email protected]>
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.

2 participants