feat(local exec): add ability to skip steps #1470
reviewdog.yml
on: pull_request
diff-review
2m 14s
full-review
2m 22s
Annotations
20 errors
|
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/pipeline/exec.go#L155
unnecessary trailing newline (whitespace)
|
|
diff-review:
action/pipeline/exec.go#L135
unnecessary leading newline (whitespace)
|
|
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/exec.go#L129
block should not start with a whitespace (wsl)
|
|
diff-review:
action/pipeline/exec.go#L165
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)
|
|
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/pipeline/exec.go#L155
unnecessary trailing newline (whitespace)
|
|
full-review:
action/pipeline/exec.go#L135
unnecessary leading newline (whitespace)
|
|
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/exec.go#L129
block should not start with a whitespace (wsl)
|
|
full-review:
action/pipeline/exec.go#L165
assignments should only be cuddled with other assignments (wsl)
|
|
full-review:
action/pipeline/validate_test.go#L487
if statements should only be cuddled with assignments (wsl)
|
|
full-review:
action/pipeline/validate_test.go#L482
assignments should only be cuddled with other assignments (wsl)
|