chore: modernize code #1270
jkroepkeAnnotations
11 errors and 2 warnings
|
lint:
internal/oauth2/provider.go#L22
argument-limit: maximum number of arguments per function exceeded; max 6 but got 7 (revive)
|
|
lint:
internal/oauth2/provider.go#L110
argument-limit: maximum number of arguments per function exceeded; max 6 but got 7 (revive)
|
|
lint:
internal/oauth2/handler.go#L84
modifies-value-receiver: suspicious assignment to a by-value method receiver (revive)
|
|
lint:
internal/httphandler/handler.go#L14
import-shadowing: The name 'oauth2' shadows an import name (revive)
|
|
lint:
internal/utils/testutils/main.go#L316
context-as-argument: context.Context should be the first parameter of a function (revive)
|
|
lint:
internal/httpserver/main.go#L72
Function `Listen$2->serve` should pass the context parameter (contextcheck)
|
|
lint:
internal/openvpn/client.go#L178
Non-inherited new context, use function like `context.WithXXX` instead (contextcheck)
|
|
lint:
cmd/daemon/root.go#L31
calculated cyclomatic complexity for function Execute is 22, max is 10 (cyclop)
|
|
lint:
internal/oauth2/handler.go#L195
WarnContext should be used instead (sloglint)
|
|
lint:
cmd/daemon/root.go#L66
SA4006: this value of `err` is never used (staticcheck)
|
|
Build & Test
Process completed with exit code 1.
|
|
lint
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
|
Build & Test
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|