fix(secrets): abstract populateEvents for secret usage #1248
reviewdog.yml
on: pull_request
diff-review
2m 16s
full-review
2m 40s
Annotations
16 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/pipeline/validate_test.go#L450
if statements should only be cuddled with assignments (wsl)
|
|
diff-review:
action/pipeline/validate_test.go#L445
assignments should only be cuddled with other assignments (wsl)
|
|
diff-review:
action/pipeline/validate_test.go#L446
only one cuddle assignment allowed before if statement (wsl)
|
|
diff-review:
command/secret/add.go#L150
directive `//nolint:dupl // ignore similar code with update` is unused for linter "dupl" (nolintlint)
|
|
diff-review:
command/secret/update.go#L150
directive `//nolint:dupl // ignore similar code with add` is unused for linter "dupl" (nolintlint)
|
|
diff-review:
command/worker/add.go#L3
directive `//nolint:dupl // ignore similar code with update` is unused for linter "dupl" (nolintlint)
|
|
diff-review:
command/worker/view.go#L70
directive `//nolint:dupl // ignore similar code with chown, get, remove and repair` is unused for linter "dupl" (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/pipeline/validate_test.go#L445
assignments should only be cuddled with other assignments (wsl)
|
|
full-review:
action/pipeline/validate_test.go#L446
only one cuddle assignment allowed before if statement (wsl)
|
|
full-review:
action/pipeline/validate_test.go#L450
if statements should only be cuddled with assignments (wsl)
|
|
full-review:
command/secret/add.go#L150
directive `//nolint:dupl // ignore similar code with update` is unused for linter "dupl" (nolintlint)
|
|
full-review:
command/secret/update.go#L150
directive `//nolint:dupl // ignore similar code with add` is unused for linter "dupl" (nolintlint)
|
|
full-review:
command/worker/add.go#L3
directive `//nolint:dupl // ignore similar code with update` is unused for linter "dupl" (nolintlint)
|
|
full-review:
command/worker/view.go#L70
directive `//nolint:dupl // ignore similar code with chown, get, remove and repair` is unused for linter "dupl" (nolintlint)
|