Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(core/rawdb): InspectDatabase uses libevm (8) #791

Open
wants to merge 3 commits into
base: libevm
Choose a base branch
from

Conversation

qdm12
Copy link
Collaborator

@qdm12 qdm12 commented Feb 12, 2025

  • Part of the core/rawdb migration to use libevm
  • Using libevm InspectDatabase
  • Define options passed to libevm InspectDatabase

@qdm12 qdm12 force-pushed the qdm12/core/rawdb/inspect-database branch from 6c4bbc2 to 1d24024 Compare February 12, 2025 11:59
@qdm12 qdm12 force-pushed the qdm12/core/rawdb/inspect-database branch from 2fc3ab9 to eafaffd Compare February 12, 2025 12:24
@qdm12 qdm12 force-pushed the qdm12/libevm-phase-2.5 branch 2 times, most recently from d311ae1 to fe9ce00 Compare February 12, 2025 12:53
@qdm12 qdm12 force-pushed the qdm12/core/rawdb/inspect-database branch from eafaffd to e6470d4 Compare February 12, 2025 17:13
@qdm12 qdm12 changed the title chore(core/rawdb): InspectDatabase uses libevm chore(core/rawdb): InspectDatabase uses libevm (2) Feb 13, 2025
@qdm12 qdm12 changed the title chore(core/rawdb): InspectDatabase uses libevm (2) chore(core/rawdb): InspectDatabase uses libevm (6) Feb 14, 2025
@qdm12 qdm12 changed the base branch from qdm12/libevm-phase-2.5 to qdm12/core/types/body-libevm February 14, 2025 13:38
@qdm12 qdm12 changed the title chore(core/rawdb): InspectDatabase uses libevm (6) chore(core/rawdb): InspectDatabase uses libevm (7) Feb 14, 2025
@qdm12 qdm12 changed the base branch from qdm12/core/types/body-libevm to qdm12/core/rawdb/inspect-database-test February 14, 2025 13:43
@qdm12 qdm12 force-pushed the qdm12/core/rawdb/inspect-database-test branch from 59d9f05 to 6a5b763 Compare February 14, 2025 13:58
@qdm12 qdm12 force-pushed the qdm12/core/rawdb/inspect-database branch from e6470d4 to 710c622 Compare February 14, 2025 13:59
@qdm12 qdm12 force-pushed the qdm12/core/rawdb/inspect-database-test branch from 6a5b763 to 1aa5bea Compare February 14, 2025 14:19
@qdm12 qdm12 force-pushed the qdm12/core/rawdb/inspect-database branch from 710c622 to 0b398d6 Compare February 14, 2025 14:20
@qdm12 qdm12 changed the title chore(core/rawdb): InspectDatabase uses libevm (7) chore(core/rawdb): InspectDatabase uses libevm (8) Feb 19, 2025
@qdm12 qdm12 mentioned this pull request Feb 21, 2025
1 task
@qdm12 qdm12 force-pushed the qdm12/core/rawdb/inspect-database-test branch from 1aa5bea to e2333a2 Compare February 28, 2025 16:42
Base automatically changed from qdm12/core/rawdb/inspect-database-test to libevm February 28, 2025 18:44
- Using libevm `InspectDatabase`
- Define options passed to libevm `InspectDatabase`
@qdm12 qdm12 force-pushed the qdm12/core/rawdb/inspect-database branch from 0b398d6 to 67a9d12 Compare February 28, 2025 18:47
@qdm12 qdm12 marked this pull request as ready for review February 28, 2025 18:48
@qdm12 qdm12 requested review from ceyonur, darioush and a team as code owners February 28, 2025 18:48
darioush
darioush previously approved these changes Feb 28, 2025
@qdm12 qdm12 force-pushed the qdm12/core/rawdb/inspect-database branch from 7fe0168 to 26ccd5d Compare February 28, 2025 19:20
Copy link
Contributor

@ARR4N ARR4N left a comment

Choose a reason for hiding this comment

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

You can really simplify this by converting it to a table-driven approach as you're applying identical logic to 4 sets of parameters. Feel free to use that commit or merge as is if you prefer.

Tip

If you want to use it you can easily fast-forward with git merge --ff-only origin/arr4n/core/rawdb/inspect-database from your branch.

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.

4 participants