chore: use go 1.23 and fix validate #1438
Triggered via pull request
September 19, 2024 16:28
Status
Success
Total duration
2m 31s
Artifacts
–
reviewdog.yml
on: pull_request
diff-review
2m 18s
full-review
2m 19s
Annotations
18 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/config/load.go#L203
assignments should only be cuddled with other assignments (wsl)
|
|
diff-review:
action/config/load.go#L204
only one cuddle assignment allowed before if statement (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/pipeline/validate_test.go#L482
assignments should only be cuddled with other assignments (wsl)
|
|
diff-review:
action/secret/table.go#L51
directive `//nolint:gosec // ignore memory aliasing` is unused for linter "gosec" (nolintlint)
|
|
diff-review:
action/secret/table.go#L106
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/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)
|
|
full-review:
action/secret/table.go#L106
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/config/load.go#L203
assignments should only be cuddled with other assignments (wsl)
|