Skip to content

ci: lint workflow files with actionlint - #892

Open
jessealama wants to merge 1 commit into
leanprover:mainfrom
jessealama:ci/actionlint
Open

ci: lint workflow files with actionlint#892
jessealama wants to merge 1 commit into
leanprover:mainfrom
jessealama:ci/actionlint

Conversation

@jessealama

Copy link
Copy Markdown
Contributor

Adds a workflow that runs actionlint over the .github/workflows directory whenever those files change. actionlint checks expression types (used in the little GitHub Action DSL), undefined step/output references and needs/if conditions, and runs shellcheck over every run: shell block and pyflakes over shell: python chunks.

To make sure that adding this workflow doesn't immediately break the build, I did a manual one-off run and found a couple of things, which I've just fixed here:

  • an unquoted $GITHUB_ENV in lean_action_ci.yml
  • an echo | sed pipeline that is now a parameter expansion

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.

1 participant