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

GH Actions: add actionlint job #925

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

jrfnl
Copy link
Member

@jrfnl jrfnl commented Jan 22, 2025

While GH Actions will fail runs of scripts which contain syntax errors, it isn't always that straight forward to see what's wrong.

The actionlint package helps with that by providing more informative error messages. Actionlint also checks for a number of best practice to help keep the GH Actions scripts secure and working.

Ref: https://github.com/rhysd/actionlint

jrfnl added 2 commits January 21, 2025 22:14
While GH Actions will fail runs of scripts which contain syntax errors, it isn't always that straight forward to see what's wrong.

The `actionlint` package helps with that by providing more informative error messages. Actionlint also checks for a number of best practice to help keep the GH Actions scripts secure and working.

Ref: https://github.com/rhysd/actionlint
Various tweaks to allow the scripts to pass the actionlint and shellcheck checks.
@jrfnl jrfnl added this to the 2.0.x Next milestone Jan 22, 2025
@jrfnl jrfnl requested a review from schlessera January 22, 2025 09:52
@jrfnl jrfnl force-pushed the feature/ghactions-add-actionlint branch from 0bea12c to 4f059f5 Compare January 22, 2025 09:52
@jrfnl jrfnl mentioned this pull request Jan 22, 2025
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant