-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Introduce actionlint job for linting GitHub Actions workflows #111
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets' try this out
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion:
if: ${{ github.event_name == 'pull_request' }}
This is a suggestion to address #84, using the reusable
lint-workflows
workflow, which under the hood is based on actionlint.I kept the workflow trigger the same as for code quality.