Merge pull request #51 from reviewdog/depup/reviewdog/reviewdog #100
haya14busareviewdog.yml
on: push
runner / shellcheck
5s
runner / shfmt
4s
runner / actionlint
6s
runner / misspell
14s
runner / alex
25s
Annotations
2 errors, 3 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.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.
|
|
runner / shellcheck
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: haya14busa/action-cond@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
[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 `...`.
|