feat(local exec): add ability to skip steps #1505
KellyMerrickreviewdog.yml
on: pull_request
diff-review
2m 36s
full-review
2m 33s
Annotations
19 errors and 2 warnings
|
diff-review:
action/pipeline/exec.go#L28
cyclomatic complexity 31 of func `(*Config).Exec` is high (> 30) (gocyclo)
|
|
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:
action/pipeline/exec.go#L28
cyclomatic complexity 31 of func `(*Config).Exec` is high (> 30) (gocyclo)
|
|
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:
action/pipeline/exec.go#L28
cyclomatic complexity 31 of func `(*Config).Exec` is high (> 30) (gocyclo)
|
|
diff-review
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
|
full-review
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|