Skip to content

chore(evm): remove unneeded lifetimes#9583

Merged
shekhirin merged 2 commits into
mainfrom
alexey/evm-lifetimes
Jul 17, 2024
Merged

chore(evm): remove unneeded lifetimes#9583
shekhirin merged 2 commits into
mainfrom
alexey/evm-lifetimes

Conversation

@shekhirin
Copy link
Copy Markdown
Member

I don't think we need them on self

@shekhirin shekhirin added C-debt A clean up/refactor of existing code A-execution-reth Related to the Execution and EVM labels Jul 17, 2024
Copy link
Copy Markdown
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.

lgtm, if this works

@shekhirin shekhirin marked this pull request as ready for review July 17, 2024 13:23
@shekhirin shekhirin requested review from Rjected and rakita as code owners July 17, 2024 13:23
@shekhirin shekhirin enabled auto-merge July 17, 2024 13:23
@shekhirin shekhirin added this pull request to the merge queue Jul 17, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 17, 2024
@shekhirin shekhirin enabled auto-merge July 17, 2024 14:12
@shekhirin shekhirin added this pull request to the merge queue Jul 17, 2024
Merged via the queue into main with commit 269c649 Jul 17, 2024
@shekhirin shekhirin deleted the alexey/evm-lifetimes branch July 17, 2024 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-execution-reth Related to the Execution and EVM C-debt A clean up/refactor of existing code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants