Skip to content

Commit

Permalink
Update README.md (#35)
Browse files Browse the repository at this point in the history
  • Loading branch information
rajbos committed Aug 28, 2023
1 parent 8edb50b commit 28b43a5
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -71,3 +71,5 @@ If you want to hide certain warnings from shellcheck, then you can use the direc
``` shell
# shellcheck disable=code
```

In some cases the directives are not picked up (might be depending on the shell it is checking. It can then help to add the `shell: your-shell-here` specification to your workflow file. I've seen this confusion happening with PowerShell code on a Windows based runner. Shellcheck was analyzing the script of the `run` step as if it where bash. The `shell` keyword was not needed for the workflow to run, as the default shell on the Windows runner was PowerShell already. Shellcheck cannot handle that. Specifying the keyword stopped the 'errors' from being reported.

0 comments on commit 28b43a5

Please sign in to comment.