Skip to content

fix: remove unused base_fee_params_at_block function#17992

Merged
mattsse merged 7 commits intoparadigmxyz:mainfrom
leniram159:issue
Aug 22, 2025
Merged

fix: remove unused base_fee_params_at_block function#17992
mattsse merged 7 commits intoparadigmxyz:mainfrom
leniram159:issue

Conversation

@leniram159
Copy link
Contributor

addresses issue

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

just needs fmt and clippy fixes

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Aug 21, 2025
@leniram159 leniram159 requested a review from mattsse August 21, 2025 19:59
@leniram159 leniram159 changed the title Fix: Remove unused base_fee_params_at_block function fix: remove unused base_fee_params_at_block function Aug 21, 2025
@leniram159
Copy link
Contributor Author

just needs fmt and clippy fixes

looks like the CI is happy now

@mattsse mattsse added this pull request to the merge queue Aug 22, 2025
Merged via the queue into paradigmxyz:main with commit d8e8d67 Aug 22, 2025
41 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Aug 22, 2025
lwedge99 pushed a commit to sentioxyz/reth that referenced this pull request Sep 16, 2025
Co-authored-by: Dharm Singh <dharmhsing@gmail.com>
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 22, 2026
…h#17992)

Co-authored-by: Dharm Singh <dharmhsing@gmail.com>
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Feb 11, 2026
…h#17992)

Co-authored-by: Dharm Singh <dharmhsing@gmail.com>
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants