fix(templates): prevent nil panic #6332
Triggered via pull request
September 26, 2024 19:09
Status
Success
Total duration
2m 18s
Artifacts
–
reviewdog.yml
on: pull_request
diff-review
2m 7s
full-review
1m 55s
Annotations
20 errors
full-review:
mock/server/authentication.go#L84
string `error` has 4 occurrences, make it a constant (goconst)
|
full-review:
database/types/build.go#L375
G115: integer overflow conversion int -> int32 (gosec)
|
full-review:
database/types/build.go#L376
G115: integer overflow conversion int -> int32 (gosec)
|
full-review:
database/types/repo.go#L336
G115: integer overflow conversion int -> int32 (gosec)
|
full-review:
database/types/settings.go#L134
G115: integer overflow conversion int64 -> uint64 (gosec)
|
full-review:
database/types/settings.go#L205
G115: integer overflow conversion uint64 -> int64 (gosec)
|
full-review:
secret/vault/delete.go#L16
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
full-review:
secret/vault/get.go#L18
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
|
full-review:
secret/vault/refresh_test.go#L45
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
|
full-review:
secret/vault/refresh_test.go#L71
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
|
diff-review:
mock/server/authentication.go#L84
string `error` has 4 occurrences, make it a constant (goconst)
|
diff-review:
database/types/build.go#L375
G115: integer overflow conversion int -> int32 (gosec)
|
diff-review:
database/types/build.go#L376
G115: integer overflow conversion int -> int32 (gosec)
|
diff-review:
database/types/repo.go#L336
G115: integer overflow conversion int -> int32 (gosec)
|
diff-review:
database/types/settings.go#L134
G115: integer overflow conversion int64 -> uint64 (gosec)
|
diff-review:
database/types/settings.go#L205
G115: integer overflow conversion uint64 -> int64 (gosec)
|
diff-review:
secret/vault/refresh_test.go#L45
unused-parameter: parameter 'r' seems to be unused, consider removing or renaming it as _ (revive)
|
diff-review:
secret/vault/refresh_test.go#L71
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
|
diff-review:
secret/vault/refresh_test.go#L108
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
|
diff-review:
secret/vault/refresh_test.go#L128
unused-parameter: parameter 'in' seems to be unused, consider removing or renaming it as _ (revive)
|