chore: v0.25.0 #1453
wass3rw3rkreviewdog.yml
on: pull_request
diff-review
2m 27s
full-review
2m 30s
Annotations
20 errors
|
diff-review:
internal/output/yaml_test.go#L72
unused-parameter: parameter 'unmarshal' seems to be unused, consider removing or renaming it as _ (revive)
|
|
diff-review:
action/hook/table.go#L20
SA1019: library.Hook is deprecated: use Hook from github.com/go-vela/server/api/types instead. (staticcheck)
|
|
diff-review:
action/hook/table.go#L69
SA1019: library.Hook is deprecated: use Hook from github.com/go-vela/server/api/types instead. (staticcheck)
|
|
diff-review:
action/hook/table.go#L118
SA1019: library.Hook is deprecated: use Hook from github.com/go-vela/server/api/types instead. (staticcheck)
|
|
diff-review:
action/config/load.go#L204
only one cuddle assignment allowed before if statement (wsl)
|
|
diff-review:
action/config/load.go#L203
assignments should only be cuddled with other assignments (wsl)
|
|
diff-review:
action/pipeline/validate_test.go#L482
assignments should only be cuddled with other assignments (wsl)
|
|
diff-review:
action/pipeline/validate_test.go#L483
only one cuddle assignment allowed before if statement (wsl)
|
|
diff-review:
action/pipeline/validate_test.go#L487
if statements should only be cuddled with assignments (wsl)
|
|
diff-review:
action/secret/table.go#L51
directive `//nolint:gosec // ignore memory aliasing` is unused for linter "gosec" (nolintlint)
|
|
full-review:
internal/output/yaml_test.go#L72
unused-parameter: parameter 'unmarshal' seems to be unused, consider removing or renaming it as _ (revive)
|
|
full-review:
action/hook/table.go#L20
SA1019: library.Hook is deprecated: use Hook from github.com/go-vela/server/api/types instead. (staticcheck)
|
|
full-review:
action/hook/table.go#L69
SA1019: library.Hook is deprecated: use Hook from github.com/go-vela/server/api/types instead. (staticcheck)
|
|
full-review:
action/hook/table.go#L118
SA1019: library.Hook is deprecated: use Hook from github.com/go-vela/server/api/types instead. (staticcheck)
|
|
full-review:
action/config/load.go#L203
assignments should only be cuddled with other assignments (wsl)
|
|
full-review:
action/config/load.go#L204
only one cuddle assignment allowed before if statement (wsl)
|
|
full-review:
action/pipeline/validate_test.go#L482
assignments should only be cuddled with other assignments (wsl)
|
|
full-review:
action/pipeline/validate_test.go#L483
only one cuddle assignment allowed before if statement (wsl)
|
|
full-review:
action/pipeline/validate_test.go#L487
if statements should only be cuddled with assignments (wsl)
|
|
full-review:
action/secret/table.go#L51
directive `//nolint:gosec // ignore memory aliasing` is unused for linter "gosec" (nolintlint)
|