Skip to content
This repository was archived by the owner on Jan 12, 2026. It is now read-only.

feat: respect miner_setGasLimit#18

Merged
haardikk21 merged 2 commits intorelease_upstreamfrom
set-gas-limit
Nov 6, 2025
Merged

feat: respect miner_setGasLimit#18
haardikk21 merged 2 commits intorelease_upstreamfrom
set-gas-limit

Conversation

@haardikk21
Copy link

📝 Summary

Similar changes as paradigmxyz/reth#19247 to respect miner_setGasLimit in both standard and flashblocks builders

💡 Motivation and Context


✅ I have completed the following steps:

  • Run make lint
  • Run make test
  • Added tests (if applicable)

@haardikk21 haardikk21 merged commit 21194b3 into release_upstream Nov 6, 2025
1 of 5 checks passed
@haardikk21 haardikk21 deleted the set-gas-limit branch November 6, 2025 14:38
danyalprout added a commit that referenced this pull request Nov 14, 2025
…it + reth 1.9.1) (flashbots#316)

* chore: Rbuilder updates for jovian hardfork (#16)

* Jovian ready op-rbuilder

* Add factoring in da footprint scalar

* Bump reth to 1.9.0

* feat: respect miner_setGasLimit (#18)

* respect miner_setGasLimit

* fmt

* chore: bump to reth 1.9.1 (#20)

* wip: telemetry resolution

* done

* fix: Add gas_limit_config field and make block_gas_limit() public

- Added gas_limit_config initialization in flashblocks context
- Changed block_gas_limit() visibility from pub(super) to pub for flashtestations access
- Removed unused Events import

Note: Tests currently fail due to rollup-boost dependency version mismatch (op-alloy 0.20.0 vs 0.22.0)

* chore: Update rollup-boost to v0.7.8 for reth 1.9.1 compatibility

- Updated rollup-boost dependency from rev dd12e8e to tag v0.7.8
- Resolves op-alloy version mismatch (0.20.0 -> 0.22.0)
- All tests now pass (94/94 passing)

* chore: apply nightly formatting fixes

Applied formatting fixes from cargo +nightly fmt to ensure code
passes lint checks. Changes include proper brace placement and
line formatting in flashblocks builder_tx and payload_handler.

---------

Co-authored-by: Danyal Prout <git@dany.al>
Co-authored-by: Haardik <hhaardik@uwaterloo.ca>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants