Skip to content

chore: modernize code #1272

chore: modernize code

chore: modernize code #1272

Triggered via pull request December 21, 2024 22:27
@jkroepkejkroepke
synchronize #378
refactor
Status Failure
Total duration 1m 11s
Artifacts

ci.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

8 errors and 2 warnings
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
issues found
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