Skip to content

Merge pull request #58 from reviewdog/depup/reviewdog/reviewdog #105

Merge pull request #58 from reviewdog/depup/reviewdog/reviewdog

Merge pull request #58 from reviewdog/depup/reviewdog/reviewdog #105

Triggered via push June 17, 2024 14:50
Status Success
Total duration 31s
Artifacts

reviewdog.yml

on: push
runner / shellcheck
10s
runner / shellcheck
runner / shfmt
5s
runner / shfmt
runner / actionlint
9s
runner / actionlint
runner / misspell
21s
runner / misspell
runner / alex
23s
runner / alex
Fit to window
Zoom out
Zoom in

Annotations

2 errors, 2 warnings, and 2 notices
[shellcheck] testdata/test.sh#L1 <ShellCheck.SC2148>: testdata/test.sh#L1
Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive.
[shellcheck] testdata/test.sh#L5 <ShellCheck.SC2283>: testdata/test.sh#L5
Remove spaces around = to assign (or use [ ] to compare, or quote '=' if literal).
[shellcheck] testdata/test.sh#L1 <ShellCheck.SC2035>: testdata/test.sh#L1
Use ./*glob* or -- *glob* so names with dashes won't become options.
[shellcheck] testdata/test.sh#L7 <ShellCheck.SC2006>: testdata/test.sh#L7
Use $(...) notation instead of legacy backticks `...`.
[shellcheck] testdata/test.sh#L1 <ShellCheck.SC2061>: testdata/test.sh#L1
Quote the parameter to -name so the shell won't interpret it.
[shellcheck] testdata/test.sh#L3 <ShellCheck.SC2063>: testdata/test.sh#L3
Grep uses regex, but this looks like a glob.