mirror of
https://github.com/netbirdio/netbird.git
synced 2024-11-25 09:33:24 +01:00
d1a323fa9d
* Add gocritic linter `gocritic` provides diagnostics that check for bugs, performance, and style issues We disable the following checks: - commentFormatting - captLocal - deprecatedComment This PR contains many `//nolint:gocritic` to disable `appendAssign`.
84 lines
3.3 KiB
YAML
84 lines
3.3 KiB
YAML
run:
|
|
# Timeout for analysis, e.g. 30s, 5m.
|
|
# Default: 1m
|
|
timeout: 6m
|
|
|
|
# This file contains only configs which differ from defaults.
|
|
# All possible options can be found here https://github.com/golangci/golangci-lint/blob/master/.golangci.reference.yml
|
|
linters-settings:
|
|
errcheck:
|
|
# Report about not checking of errors in type assertions: `a := b.(MyStruct)`.
|
|
# Such cases aren't reported by default.
|
|
# Default: false
|
|
check-type-assertions: false
|
|
|
|
gocritic:
|
|
disabled-checks:
|
|
- commentFormatting
|
|
- captLocal
|
|
- deprecatedComment
|
|
|
|
govet:
|
|
# Enable all analyzers.
|
|
# Default: false
|
|
enable-all: false
|
|
enable:
|
|
- nilness
|
|
|
|
tenv:
|
|
# The option `all` will run against whole test files (`_test.go`) regardless of method/function signatures.
|
|
# Otherwise, only methods that take `*testing.T`, `*testing.B`, and `testing.TB` as arguments are checked.
|
|
# Default: false
|
|
all: true
|
|
|
|
linters:
|
|
disable-all: true
|
|
enable:
|
|
## enabled by default
|
|
- errcheck # checking for unchecked errors, these unchecked errors can be critical bugs in some cases
|
|
- gosimple # specializes in simplifying a code
|
|
- govet # reports suspicious constructs, such as Printf calls whose arguments do not align with the format string
|
|
- ineffassign # detects when assignments to existing variables are not used
|
|
- staticcheck # is a go vet on steroids, applying a ton of static analysis checks
|
|
- tenv # Tenv is analyzer that detects using os.Setenv instead of t.Setenv since Go1.17.
|
|
- typecheck # like the front-end of a Go compiler, parses and type-checks Go code
|
|
- unused # checks for unused constants, variables, functions and types
|
|
## disable by default but the have interesting results so lets add them
|
|
- bodyclose # checks whether HTTP response body is closed successfully
|
|
- dupword # dupword checks for duplicate words in the source code
|
|
- durationcheck # durationcheck checks for two durations multiplied together
|
|
- forbidigo # forbidigo forbids identifiers
|
|
- gocritic # provides diagnostics that check for bugs, performance and style issues
|
|
- mirror # mirror reports wrong mirror patterns of bytes/strings usage
|
|
- misspell # misspess finds commonly misspelled English words in comments
|
|
- nilerr # finds the code that returns nil even if it checks that the error is not nil
|
|
- nilnil # checks that there is no simultaneous return of nil error and an invalid value
|
|
- predeclared # predeclared finds code that shadows one of Go's predeclared identifiers
|
|
- sqlclosecheck # checks that sql.Rows and sql.Stmt are closed
|
|
- thelper # thelper detects Go test helpers without t.Helper() call and checks the consistency of test helpers.
|
|
- wastedassign # wastedassign finds wasted assignment statements
|
|
issues:
|
|
# Maximum count of issues with the same text.
|
|
# Set to 0 to disable.
|
|
# Default: 3
|
|
max-same-issues: 5
|
|
|
|
exclude-rules:
|
|
# allow fmt
|
|
- path: management/cmd/root.go
|
|
linters: forbidigo
|
|
- path: signal/cmd/root.go
|
|
linters: forbidigo
|
|
- path: sharedsock/filter.go
|
|
linters:
|
|
- unused
|
|
- path: client/firewall/iptables/rule.go
|
|
linters:
|
|
- unused
|
|
- path: test.go
|
|
linters:
|
|
- mirror
|
|
- path: mock.go
|
|
linters:
|
|
- nilnil
|