gci order is incorrect for blank-import from local module

[ARR4N]

This should be in a separate block at the bottom.

libevm/core/vm/contracts.libevm_test.go

Line 38 in e18a1e9

_ "github.com/ava-labs/libevm/eth/tracers/native"

[qdm12]

Indeed the gci linter used doesn't seem to apply the import sections order defined, where blank imports should be the last section. On the other hand, is there a reason we would want this? Maybe to deal with various init() functions? 🤔

[ARR4N]

is there a reason we would want this?

Since underscore imports are for side effects and they're the only place in the code that you see it, I prefer to put them at the end as a way to call attention to them. It's not a high priority at all, and just put in an issue to avoid forgetting about it.