Bump golangci-lint to v1.56.1 #5311
Annotations
5 errors and 11 warnings
lint:
app_test.go#L826
go-require: require must only be used in the goroutine running the test function (testifylint)
|
lint:
app_test.go#L869
go-require: require must only be used in the goroutine running the test function (testifylint)
|
lint:
app_test.go#L1546
go-require: require must only be used in the goroutine running the test function (testifylint)
|
lint:
redirect_test.go#L149
error-is-as: second argument to require.ErrorAs should not be *error (testifylint)
|
lint
issues found
|
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
lint:
app_test.go#L239
unused-parameter: parameter 'err' seems to be unused, consider removing or renaming it as _ (revive)
|
lint:
client_test.go#L47
unused-parameter: parameter 'addr' seems to be unused, consider removing or renaming it as _ (revive)
|
lint:
client_test.go#L89
unused-parameter: parameter 'addr' seems to be unused, consider removing or renaming it as _ (revive)
|
lint:
client_test.go#L674
unused-parameter: parameter 'req' seems to be unused, consider removing or renaming it as _ (revive)
|
lint:
ctx_test.go#L3402
unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)
|
lint:
ctx_test.go#L3493
unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)
|
lint:
ctx_test.go#L3570
unused-parameter: parameter 'v' seems to be unused, consider removing or renaming it as _ (revive)
|
lint:
helpers.go#L56
max-control-nesting: control flow nesting exceeds 5 (revive)
|
lint:
helpers_test.go#L259
unused-parameter: parameter 's1' seems to be unused, consider removing or renaming it as _ (revive)
|
lint:
hooks_test.go#L91
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
|