mirror of
https://github.com/qdm12/gluetun.git
synced 2026-05-06 20:10:11 +02:00
Quentin McGaw
4a78989d9d
- main reason being it's a burden to always define sentinel errors at global scope, wrap them with `%w` instead of using a string directly - only use sentinel errors when it has to be checked using `errors.Is` - replace all usage of these sentinel errors in `fmt.Errorf` with direct strings that were in the sentinel error - exclude the sentinel error definition requirement from .golangci.yml - update unit tests to use ContainersError instead of ErrorIs so it stays as a "not a change detector test" without requiring a sentinel error
148 lines
2.8 KiB
YAML
148 lines
2.8 KiB
YAML
version: "2"
|
|
|
|
formatters:
|
|
enable:
|
|
- gci
|
|
- gofumpt
|
|
- goimports
|
|
exclusions:
|
|
generated: lax
|
|
paths:
|
|
- third_party$
|
|
- builtin$
|
|
- examples$
|
|
|
|
linters:
|
|
settings:
|
|
misspell:
|
|
locale: US
|
|
goconst:
|
|
ignore-string-values:
|
|
# commonly used settings strings
|
|
- "^disabled$"
|
|
# Firewall and routing strings
|
|
- "^(ACCEPT|DROP)$"
|
|
- "^--append$"
|
|
- "^--delete$"
|
|
- "^all$"
|
|
- "^(tcp|udp)$"
|
|
# Server route strings
|
|
- "^/status$"
|
|
|
|
exclusions:
|
|
generated: lax
|
|
presets:
|
|
- comments
|
|
- common-false-positives
|
|
- legacy
|
|
- std-error-handling
|
|
rules:
|
|
- linters:
|
|
- containedctx
|
|
- dupl
|
|
- err113
|
|
- maintidx
|
|
path: _test\.go
|
|
- linters:
|
|
- dupl
|
|
path: internal\/server\/.+\.go
|
|
- linters:
|
|
- ireturn
|
|
text: returns interface \(golang\.org\/x\/sys\/unix\.Sockaddr\)
|
|
- linters:
|
|
- ireturn
|
|
path: internal\/openvpn\/pkcs8\/descbc\.go
|
|
text: newCipherDESCBCBlock returns interface \(github\.com\/youmark\/pkcs8\.Cipher\)
|
|
- linters:
|
|
- revive
|
|
path: internal\/provider\/(common|utils)\/.+\.go
|
|
text: "var-naming: avoid (bad|meaningless) package names"
|
|
- linters:
|
|
- lll
|
|
source: "^// https://.+$"
|
|
- linters:
|
|
- mnd
|
|
source: "^ cleanups\\.Add.+$"
|
|
path: internal\/(wireguard|amneziawg)\/run\.go
|
|
- linters:
|
|
- err113
|
|
- mnd
|
|
path: ci\/.+\.go
|
|
- linters:
|
|
- err113
|
|
text: "do not define dynamic errors, use wrapped static errors instead"
|
|
|
|
paths:
|
|
- third_party$
|
|
- builtin$
|
|
- examples$
|
|
enable:
|
|
# - cyclop
|
|
# - errorlint
|
|
- asasalint
|
|
- asciicheck
|
|
- bidichk
|
|
- bodyclose
|
|
- containedctx
|
|
- copyloopvar
|
|
- decorder
|
|
- dogsled
|
|
- dupl
|
|
- dupword
|
|
- durationcheck
|
|
- err113
|
|
- errchkjson
|
|
- errname
|
|
- exhaustive
|
|
- fatcontext
|
|
- forcetypeassert
|
|
- gocheckcompilerdirectives
|
|
- gochecknoglobals
|
|
- gochecknoinits
|
|
- gocognit
|
|
- goconst
|
|
- gocritic
|
|
- gocyclo
|
|
- godot
|
|
- goheader
|
|
- gomoddirectives
|
|
- goprintffuncname
|
|
- gosec
|
|
- gosmopolitan
|
|
- grouper
|
|
- importas
|
|
- interfacebloat
|
|
- intrange
|
|
- ireturn
|
|
- lll
|
|
- maintidx
|
|
- makezero
|
|
- mirror
|
|
- misspell
|
|
- mnd
|
|
- musttag
|
|
- nakedret
|
|
- nestif
|
|
- nilerr
|
|
- nilnil
|
|
- noctx
|
|
- nolintlint
|
|
- nosprintfhostport
|
|
- paralleltest
|
|
- prealloc
|
|
- predeclared
|
|
- promlinter
|
|
- reassign
|
|
- revive
|
|
- rowserrcheck
|
|
- sqlclosecheck
|
|
- tagalign
|
|
- thelper
|
|
- tparallel
|
|
- unconvert
|
|
- unparam
|
|
- usestdlibvars
|
|
- wastedassign
|
|
- whitespace
|
|
- zerologlint
|