Skip to content

Conversation

@JonathanOppenheimer
Copy link
Member

Why this should be merged

This libevm import is not allowed and should not be used.

How this was tested

Existing UT

Need to be documented?

No

Need to update RELEASES.md?

No

@JonathanOppenheimer JonathanOppenheimer self-assigned this Oct 17, 2025
@JonathanOppenheimer JonathanOppenheimer requested a review from a team as a code owner October 17, 2025 19:20
@JonathanOppenheimer JonathanOppenheimer force-pushed the JonathanOppenheimer/remove-pseudo branch from 5308d55 to 65b6bf9 Compare October 17, 2025 19:23
@JonathanOppenheimer JonathanOppenheimer changed the title Remove libevm/psuedo import Remove libevm/psuedo import Oct 17, 2025
@JonathanOppenheimer
Copy link
Member Author

This is not necessary right now. As per @ARR4N

Although the new approach compiles, it suggests some incorrect assumptions to the reader and that's worse than allowing the import. [...] Let's leave it as is and I'll look at a broader refactor later.

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.

3 participants