Skip to content

AvalancheGo Linter integration #1144

@alarso16

Description

@alarso16

The avalanchego linter is applied on all Ava Labs owned files in this repo, but not all linters can be applied. Because it previously adhered only to a weaker linter, adding all linters would cause it to fail. Here are all the linters currently unused from .avalanche-golangci.yml:

  • depguard
  • errcheck
  • forbidigo (must do depguard first, or add handling for assert cases)
    • t.Fatal
    • require.Error and require.ErrorContains
    • Remaining
  • gocritic
  • gosec
  • nilerr
  • prealloc
  • predeclared
  • revive
  • staticcheck
  • testifylint

Each of these changes should be reflected in coreth prior to subnet-evm to keep the repos synced in properly.

Metadata

Metadata

Labels

Type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions