Skip to content

ci: add actionlint check for workflow files#5758

Merged
rene merged 1 commit intolf-edge:masterfrom
rucoder:rucoder/add-actionlint
Apr 7, 2026
Merged

ci: add actionlint check for workflow files#5758
rene merged 1 commit intolf-edge:masterfrom
rucoder:rucoder/add-actionlint

Conversation

@rucoder
Copy link
Copy Markdown
Contributor

@rucoder rucoder commented Apr 7, 2026

Description

Unfortunately Yetus doesn't catch errors in workflow files. I tested this workflow on Rene's repo

Add a GitHub Actions workflow lint check that runs on PRs touching .github/workflows/**. Uses actionlint via reviewdog to post inline review comments on workflow syntax errors

Would have caught the bug fixed in #5757.

Only triggers when workflow files are modified. Runs on ubuntu-24.04.

How to test and validate this PR

  • Create a PR with a broken workflow file — actionlint should flag it with inline review comments

Changelog notes

No user-facing changes.

PR Backports

  • 16.0-stable: No
  • 14.5-stable: No
  • 13.4-stable: No

Checklist

  • I've provided a proper description

  • I've written the test verification instructions

  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

@github-actions github-actions Bot requested a review from uncleDecart April 7, 2026 07:52
@rucoder rucoder force-pushed the rucoder/add-actionlint branch 2 times, most recently from 641374d to fbe1f47 Compare April 7, 2026 07:56
Add a GitHub Actions workflow lint check that runs on PRs touching
.github/workflows/. Uses actionlint via reviewdog to post inline
review comments on workflow syntax errors (like empty with: blocks,
invalid expressions, unknown inputs, etc.).

Only triggers when workflow files are modified.

Signed-off-by: Mikhail Malyshev <mike.malyshev@gmail.com>
@rucoder rucoder force-pushed the rucoder/add-actionlint branch from fbe1f47 to 8bac1ba Compare April 7, 2026 08:02
@rene rene merged commit e4abf88 into lf-edge:master Apr 7, 2026
17 checks passed
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