Epic: Subnet-EVM code readiness for monorepo

[JonathanOppenheimer]

The following PRs must be merged (or resolved in another manner), prior to subnet-evm being ready for the monorepo.

The below is the category of fix, and the PRs associated with it. Any and all reviews are appreciated!

Tests:

• Flaky tests Epic: fix flaky EVM tests avalanchego#4558
  • This is historically optional, as a lot of these flaky tests were introduced into AvalancheGo (through graft/coreth) without resolution.
• Skip filters test sync: coreth PR #1404: ci: Filters test corrections #1889
• Do not retry tests ci: Remove test retries #1877

Build:

• Remove go run x in favor of tooling sync: coreth PR #1407 build: remove unneeded dependencies #1888
• Remove dead t8n code chore: remove dead t8n code #1901
• Update Subnet-EVM to latest AvalancheGo version chore: Update AvalancheGo version #1896

Linting:

• Links linter sync: coreth PR #1392: chore: Add lint link check job #1870
• forbidigo sync: coreth PR #1303 style: Use require.ErrorIs whenever possible #1875
• Remove old unpack code chore: remove old pack/unpack code #1903
  • This is not critical, but resolves a linting / style concern that @alarso16 had. Right now the resultant code is nolinted
• errcheck sync: coreth PR #1203: style: enable errcheck linter #1876
• depguard sync: coreth PR #1271: style: enable depguard linter #1886
• License headers sync: coreth PR #1424: fix license header #1891
• shellcheck directives chore: fix shellcheck directives #1905

Precompile test rewrites:

• tx allowlist feat: convert tx allow list npm test to go test #1897
• fee manager feat: convert fee manager npm test to go test #1885
• reward manager feat: convert tx reward manager npm test to go test and remove e2e-precompile testing infrastructure #1900
• warp [...]

[ceyonur]

Can we also add a task to move the contracts npm package to a separate repository under subnet-evm-precompile-examples etc and archive/make readonly that repo? We should be picking a revision at the time there was no binding and where the folder was a pure hardhat/npm package. cc @JonathanOppenheimer

[JonathanOppenheimer]

Can we also add a task to move the contracts npm package to a separate repository under subnet-evm-precompile-examples etc and archive/make readonly that repo? We should be picking a revision at the time there was no binding and where the folder was a pure hardhat/npm package. cc @JonathanOppenheimer

We certainly can, but from talking Arran, he expressed that he saw little utility in doing so (creating an examples repository that is). Is this still something desired?