Skip to content

test: remove old import compliance test#4767

Merged
maru-ava merged 2 commits intomasterfrom
JonathanOppenheimer/remove-old-imports-test
Dec 18, 2025
Merged

test: remove old import compliance test#4767
maru-ava merged 2 commits intomasterfrom
JonathanOppenheimer/remove-old-imports-test

Conversation

@JonathanOppenheimer
Copy link
Member

@JonathanOppenheimer JonathanOppenheimer commented Dec 17, 2025

Why this should be merged

This test was completely redone and removed in #4537. However, I forgot to remove the subnet-evm version of the test, and just removed the old coreth version. This was an oversight and both tests needed to be removed.

How this was tested

CI

Need to be documented in RELEASES.md?

No

@JonathanOppenheimer JonathanOppenheimer self-assigned this Dec 17, 2025
@JonathanOppenheimer JonathanOppenheimer requested a review from a team as a code owner December 17, 2025 22:54
@JonathanOppenheimer JonathanOppenheimer added evm Related to EVM functionality subnet-evm Related to the former subnet-evm standalone repository labels Dec 17, 2025
@JonathanOppenheimer JonathanOppenheimer changed the title test: remove import compliance old test test: remove old import compliance test Dec 17, 2025
@JonathanOppenheimer JonathanOppenheimer added the testing This primarily focuses on testing label Dec 17, 2025
@maru-ava maru-ava added this pull request to the merge queue Dec 18, 2025
Merged via the queue into master with commit 489dcf7 Dec 18, 2025
52 checks passed
@maru-ava maru-ava deleted the JonathanOppenheimer/remove-old-imports-test branch December 18, 2025 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

evm Related to EVM functionality subnet-evm Related to the former subnet-evm standalone repository testing This primarily focuses on testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants