Merge branch 'main' into sync-fork-tmp #3
krhitesh7Annotations
10 errors and 6 warnings
|
lint:
pkg/cache/v3/delta.go#L150
SA4010: this result of append is never used, except maybe in other appends (staticcheck)
|
|
lint:
pkg/cache/v3/linear_test.go#L476
require-error: for error assertions use require (testifylint)
|
|
lint:
pkg/cache/v3/linear_test.go#L631
require-error: for error assertions use require (testifylint)
|
|
lint:
pkg/cache/v3/linear_test.go#L651
require-error: for error assertions use require (testifylint)
|
|
lint:
pkg/cache/v3/cache_test.go#L31
error-nil: use assert.NoError (testifylint)
|
|
lint:
pkg/cache/v3/cache_test.go#L33
len: use assert.Len (testifylint)
|
|
lint:
pkg/cache/v3/cache_test.go#L36
error-nil: use assert.NoError (testifylint)
|
|
lint:
pkg/cache/v3/cache_test.go#L41
error-nil: use assert.NoError (testifylint)
|
|
lint:
pkg/cache/v3/cache_test.go#L42
expected-actual: need to reverse actual and expected values (testifylint)
|
|
lint:
pkg/cache/v3/cache_test.go#L62
len: use assert.Len (testifylint)
|
|
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, 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:
internal/upstream/main.go#L35
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
pkg/server/v3/server_test.go#L373
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
pkg/server/v3/delta_test.go#L438
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
pkg/server/v3/server_test.go#L379
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
|
lint:
pkg/server/v3/server_test.go#L386
unused-parameter: parameter 'request' seems to be unused, consider removing or renaming it as _ (revive)
|