feat(pagination): add support for UUID array type (#59) #19
AlexHashgraphAnnotations
10 errors and 13 warnings
|
Lint:
pagination/psql_pagination.go#L71
importShadow: shadow of imported from 'k8s.io/utils/ptr' package 'ptr' (gocritic)
|
|
Lint:
pagination/psql_pagination.go#L179
fmt.Sprintf can be replaced with string concatenation (perfsprint)
|
|
Lint:
pagination/psql_pagination.go#L185
fmt.Sprintf can be replaced with string concatenation (perfsprint)
|
|
Lint:
pagination/psql_pagination_test.go#L409
expected-actual: need to reverse actual and expected values (testifylint)
|
|
Lint:
pagination/psql_pagination_test.go#L49
test helper function should start from t.Helper() (thelper)
|
|
Lint:
pagination/cursor.go#L81
variable name 'v' is too short for the scope of its usage (varnamelen)
|
|
Lint:
pagination/cursor.go#L70
if statements should only be cuddled with assignments (wsl)
|
|
Lint:
pagination/cursor.go#L73
if statements should only be cuddled with assignments (wsl)
|
|
Lint:
pagination/cursor.go#L76
return statements should not be cuddled if block has more than two lines (wsl)
|
|
Lint:
pagination/psql_pagination.go#L180
return statements should not be cuddled if block has more than two lines (wsl)
|
|
Test
Restore cache failed: Dependencies file is not found in /home/runner/work/go-commons/go-commons. Supported file pattern: go.sum
|
|
Lint
The following actions uses Node.js version which is deprecated and will be forced to run on node20: golangci/golangci-lint-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Lint:
pagination/cursor.go#L12
var-naming: don't use underscores in Go names; const UUID_Length should be UUIDLength (revive)
|
|
Lint:
pagination/cursor_internal_test.go#L23
unused-parameter: parameter 'i' seems to be unused, consider removing or renaming it as _ (revive)
|
|
Lint:
pagination/cursor_internal_test.go#L36
unused-parameter: parameter 'i' seems to be unused, consider removing or renaming it as _ (revive)
|
|
Lint:
pagination/cursor_internal_test.go#L49
unused-parameter: parameter 'i' seems to be unused, consider removing or renaming it as _ (revive)
|
|
Lint:
pagination/cursor_internal_test.go#L95
line-length-limit: line is 130 characters, out of limit 100 (revive)
|
|
Lint:
pagination/cursor_internal_test.go#L107
line-length-limit: line is 129 characters, out of limit 100 (revive)
|
|
Lint:
pagination/cursor_internal_test.go#L120
line-length-limit: line is 132 characters, out of limit 100 (revive)
|
|
Lint:
pagination/cursor_internal_test.go#L132
line-length-limit: line is 126 characters, out of limit 100 (revive)
|
|
Lint:
pagination/cursor_internal_test.go#L196
line-length-limit: line is 125 characters, out of limit 100 (revive)
|
|
Lint:
pagination/cursor_internal_test.go#L201
line-length-limit: line is 125 characters, out of limit 100 (revive)
|
|
Lint
Restore cache failed: Dependencies file is not found in /home/runner/work/go-commons/go-commons. Supported file pattern: go.sum
|