Skip to content
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

Lint with all workflows with actionlint #970

Merged
merged 3 commits into from
Apr 5, 2024
Merged

Conversation

klutchell
Copy link
Collaborator

@klutchell klutchell commented Apr 4, 2024

Alternative to #968 in order to combine actionlint with pre-commit hooks.

Repositories with workflows containing linting errors will fail the check and see annotations in the changed files tab.

For now shellcheck linting has been disabled, and will be enabled in a future PR.

@klutchell klutchell changed the title Apply recommendations from rhysd/actionlint Lint with rhysd/actionlint Apr 4, 2024
@klutchell klutchell force-pushed the kyle/actionlint branch 2 times, most recently from 14a5b85 to ee399c4 Compare April 4, 2024 20:19
@flowzone-app flowzone-app bot merged commit 9bf96c1 into master Apr 5, 2024
74 checks passed
@flowzone-app flowzone-app bot deleted the kyle/actionlint branch April 5, 2024 16:38
@klutchell klutchell mentioned this pull request Apr 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants