You've already forked oauth2-proxy
mirror of
https://github.com/oauth2-proxy/oauth2-proxy.git
synced 2025-08-06 22:42:56 +02:00
* upgrade to go1.24.5 dependency updates lint fixes chore(deps): upgrade github.com/spf13/viper to v1.20.1 Note that this upgrade also implied to upgrade github.com/mitchellh/mapstructure (nowadays unmaintained: https://gist.github.com/mitchellh/90029601268e59a29e64e55bab1c5bdc) to github.com/go-viper/mapstructure/v2. fix: adapt tests to match mapstructure v2 error messages pkg/apis/options/load_test.go: skip tests on Go 1.23 Add a compile guard for Go < 1.24 for the pkg/apis/options/load_test.go because the LoadYAML test depends on error messages produced by encoding/json that changed slightly (names of embedded structs are now reported). As we updated the test for go1.24, the test now fails on 1.23, but just for a slight difference, so we disable the test there. fix: adapt tests to match mapstructure v2 error messages remove pre 1.24 disclaimer add changelog entry Signed-off-by: Jan Larwig <jan@larwig.com> Co-Authored-By: Olivier Mengué <dolmen@cpan.org> * add exclusion for 'avoid meaningless package names' in .golangci.yml * chore(dep): upgrade all dependencies Signed-off-by: Jan Larwig <jan@larwig.com> --------- Signed-off-by: Jan Larwig <jan@larwig.com> Co-authored-by: Olivier Mengué <dolmen@cpan.org> Co-authored-by: Jan Larwig <jan@larwig.com>
70 lines
1.4 KiB
YAML
70 lines
1.4 KiB
YAML
version: "2"
|
|
linters:
|
|
default: none
|
|
enable:
|
|
- bodyclose
|
|
- copyloopvar
|
|
- dogsled
|
|
- goconst
|
|
- gocritic
|
|
- goprintffuncname
|
|
- gosec
|
|
- govet
|
|
- ineffassign
|
|
- misspell
|
|
- prealloc
|
|
- revive
|
|
- staticcheck
|
|
- unconvert
|
|
- unused
|
|
exclusions:
|
|
generated: lax
|
|
presets:
|
|
- comments
|
|
- common-false-positives
|
|
- legacy
|
|
- std-error-handling
|
|
rules:
|
|
- linters:
|
|
- bodyclose
|
|
- goconst
|
|
- gocritic
|
|
- gosec
|
|
- revive
|
|
- scopelint
|
|
- unconvert
|
|
path: _test\.go
|
|
- linters:
|
|
- revive
|
|
path: _test\.go
|
|
text: 'dot-imports:'
|
|
# # If we have tests in shared test folders, these can be less strictly linted
|
|
- linters:
|
|
- bodyclose
|
|
- revive
|
|
- staticcheck
|
|
path: tests/.*_tests\.go
|
|
# See https://github.com/oauth2-proxy/oauth2-proxy/issues/3060
|
|
# https://staticcheck.dev/docs/checks/#QF1008
|
|
- linters:
|
|
- staticcheck
|
|
text: QF1008
|
|
- linters:
|
|
- revive
|
|
path: util/.*\.go$
|
|
text: "var-naming: avoid meaningless package names"
|
|
paths:
|
|
- third_party$
|
|
- builtin$
|
|
- examples$
|
|
formatters:
|
|
enable:
|
|
- gofmt
|
|
- goimports
|
|
exclusions:
|
|
generated: lax
|
|
paths:
|
|
- third_party$
|
|
- builtin$
|
|
- examples$
|