ipn/wg: s/loged/logev #899
Annotations
18 errors, 12 warnings, and 1 notice
🧭 Lint:
intra/common.go#L595
func firstEmpty is unused (U1000)
|
🧭 Lint:
intra/udp.go#L53
var errNoProxyID is unused (U1000)
|
🧭 Lint
Process completed with exit code 1.
|
🧭 Lint:
intra/bootstrap.go#L179
Error return value of `prev.Stop` is not checked (errcheck)
|
🧭 Lint:
intra/common.go#L595
func `firstEmpty` is unused (unused)
|
🧭 Lint:
intra/dialers/direct_split.go#L31
`strat` is a misspelling of `start` (misspell)
|
🧭 Lint:
intra/dialers/retrier.go#L155
`strat` is a misspelling of `start` (misspell)
|
🧭 Lint:
intra/dialers/retrier.go#L156
`Strat` is a misspelling of `Start` (misspell)
|
🧭 Lint:
intra/dialers/retrier.go#L158
`Strat` is a misspelling of `Start` (misspell)
|
🧭 Lint:
intra/dialers/retrier.go#L178
`strat` is a misspelling of `start` (misspell)
|
🧭 Lint:
intra/dialers/retrier.go#L184
`strat` is a misspelling of `start` (misspell)
|
🧭 Lint:
intra/dialers/retrier.go#L188
`strat` is a misspelling of `start` (misspell)
|
🧭 Lint:
intra/dialers/retrier.go#L190
`strat` is a misspelling of `start` (misspell)
|
🧬 Build:
intra/udp.go#L327
error: Potential nil panic detected. Observed nil flow from source to dereference point:
|
🧬 Build
result 0 of `ProxyTo()` lacking guarding; called `ID()` via the assignment(s):
|
🧬 Build:
intra/udp.go#L327
error: Potential nil panic detected. Observed nil flow from source to dereference point:
|
🧬 Build
unassigned variable `px` called `ID()`
|
🧬 Build
Process completed with exit code 3.
|
🧭 Lint
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4, golangci/[email protected]. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
🧭 Lint:
intra/backend/core_iptree.go#L7
package-comments: should have a package comment (revive)
|
🧭 Lint:
intra/backend/core_iptree.go#L21
var-naming: type IpTree should be IPTree (revive)
|
🧭 Lint:
intra/backend/core_iptree.go#L77
var-naming: func NewIpTree should be NewIPTree (revive)
|
🧭 Lint:
intra/backend/core_iptree.go#L96
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
|
🧭 Lint:
intra/backend/core_iptree.go#L146
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
|
🧭 Lint:
intra/backend/core_iptree.go#L231
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
|
🧭 Lint:
intra/backend/core_iptree.go#L279
unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)
|
🧭 Lint:
intra/backend/core_iptree.go#L298
unused-parameter: parameter 'k' seems to be unused, consider removing or renaming it as _ (revive)
|
🧭 Lint:
intra/backend/core_iptree.go#L339
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
|
🧭 Lint:
intra/backend/core_iptree.go#L361
indent-error-flow: if block ends with a return statement, so drop this else and outdent its block (move short variable declaration to its own line if necessary) (revive)
|
🧬 Build
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
🧬 Build
success
|