Introduce actionlint job for linting GitHub Actions workflows#111
Introduce actionlint job for linting GitHub Actions workflows#111swissspidy merged 3 commits intowp-cli:mainfrom tfrommen:patch-1
Conversation
|
Thanks for the PR, @tfrommen! I don't think we'd want to use Also, if we move this to |
|
Hi @swissspidy, happy to change things. What should it be? |
|
After sleeping on it, let's go with the suggested integration as per the project's examples, and do that in |
|
@swissspidy done. I added the |
I get the impression that shellcheck is now disabled instead... ? From Flags:
-shellcheck string
Command name or file path of "shellcheck" external command. If empty, shellcheck integration will be disabled (default "shellcheck") |
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Add problem matcher | ||
| run: | |
There was a problem hiding this comment.
Suggestion:
if: ${{ github.event_name == 'pull_request' }}
This is a suggestion to address #84, using the reusable
lint-workflowsworkflow, which under the hood is based on actionlint.I kept the workflow trigger the same as for code quality.