Skip to content

refactor: remove legacy package #302

refactor: remove legacy package

refactor: remove legacy package #302

Triggered via pull request April 2, 2024 17:57
Status Failure
Total duration 53s
Artifacts

build.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 10 warnings
coverage-threshold: internal/vpntest/packetio.go#L6
package slices is not in GOROOT (/opt/hostedtoolcache/go/1.20.14/x64/src/slices)
coverage-threshold
Process completed with exit code 2.
short-tests: internal/vpntest/packetio.go#L6
package slices is not in GOROOT (/opt/hostedtoolcache/go/1.20.14/x64/src/slices)
short-tests
Process completed with exit code 1.
lint: internal/networkio/common_test.go#L27
parameter 'ctx' seems to be unused, consider removing or renaming it as _
lint: internal/mocks/addr.go#L1
should have a package comment
lint: internal/networkio/common_test.go#L27
parameter 'address' seems to be unused, consider removing or renaming it as _
lint: internal/networkio/common_test.go#L27
parameter 'network' seems to be unused, consider removing or renaming it as _
lint: internal/runtimex/runtimex_test.go#L13
parameter 't' seems to be unused, consider removing or renaming it as _
lint: internal/networkio/networkio_test.go#L83
parameter 'ctx' seems to be unused, consider removing or renaming it as _
lint: internal/session/manager.go#L114
comment on exported method Manager.NewACKForPacketIDs should be of the form "NewACKForPacketIDs ..."
lint: internal/networkio/networkio_test.go#L83
parameter 'address' seems to be unused, consider removing or renaming it as _
lint: internal/runtimex/runtimex_test.go#L22
parameter 't' seems to be unused, consider removing or renaming it as _
lint: internal/tlssession/tlshandshake_test.go#L456
error var tlsFactoryError should have name of the form errFoo