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(all): minimal changes for last libevm dependency (1) #785

Closed
wants to merge 5 commits into from

Conversation

qdm12
Copy link
Collaborator

@qdm12 qdm12 commented Feb 10, 2025

See #811 (replacement for this PR)

@qdm12 qdm12 force-pushed the qdm12/libevm-phase-2.5 branch 2 times, most recently from 17233f7 to fee6168 Compare February 10, 2025 10:54
@qdm12 qdm12 changed the title chore(all): libevm phase 2.5 chore(all): minimal changes to support the last libevm dependency Feb 10, 2025
@qdm12 qdm12 changed the title chore(all): minimal changes to support the last libevm dependency chore(all): minimal changes for last libevm dependency Feb 10, 2025
@qdm12 qdm12 force-pushed the qdm12/libevm-phase-2.5 branch 2 times, most recently from 61a477f to 75b9e0d Compare February 10, 2025 14:30
- Uses tip of main libevm dependency + cherry picked commit: ava-labs/libevm@d3f6a5e
- Precompile legacy gas fix (done in ava-labs/libevm@f906679)
- Added `HeaderExtra` not-implemented implementation
- Minimal changes shamelessly copied over from #746
@qdm12 qdm12 force-pushed the qdm12/libevm-phase-2.5 branch from 75b9e0d to ab19679 Compare February 10, 2025 14:40
@qdm12 qdm12 marked this pull request as ready for review February 10, 2025 14:53
@qdm12 qdm12 requested review from ceyonur, darioush and a team as code owners February 10, 2025 14:53
@qdm12 qdm12 changed the title chore(all): minimal changes for last libevm dependency chore(all): minimal changes for last libevm dependency (1) Feb 10, 2025
Copy link
Collaborator

@darioush darioush left a comment

Choose a reason for hiding this comment

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

LGTM once we have the tag it can be merged

@qdm12 qdm12 force-pushed the qdm12/libevm-phase-2.5 branch from 375aac8 to d311ae1 Compare February 12, 2025 12:28
@qdm12 qdm12 force-pushed the qdm12/libevm-phase-2.5 branch from d311ae1 to fe9ce00 Compare February 12, 2025 12:53
@qdm12 qdm12 force-pushed the qdm12/libevm-phase-2.5 branch from 0c36ee1 to 32a879f Compare February 14, 2025 11:01
@qdm12 qdm12 marked this pull request as draft February 19, 2025 08:22
@qdm12
Copy link
Collaborator Author

qdm12 commented Feb 19, 2025

Being replaced by #811

@qdm12
Copy link
Collaborator Author

qdm12 commented Feb 19, 2025

Replaced by #811

@qdm12 qdm12 closed this Feb 19, 2025
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