Skip to content

ci: switch actionlint to kjanat/actionlint - #710

Merged
kolyshkin merged 1 commit into
containers:mainfrom
kolyshkin:actionlint-kjanat
Oct 29, 2026
Merged

kolyshkin merged 1 commit into
containers:mainfrom
kolyshkin:actionlint-kjanat

Conversation

@kolyshkin

Copy link
Copy Markdown
Collaborator

kjanat/actionlint is an actively maintained actionlint fork that ships its own action (with built-in shellcheck), so use it directly. The fork fixes many bugs (including 2 or 3 I reported to the original repo), and knows about ubuntu-26.04.

Pin to the v1.17 tag rather than v1.17.0, as the former also pins the action's docker image by digest.

@kolyshkin
kolyshkin requested a review from jnovy October 26, 2026 07:45
devops-actions/actionlint is a third-party wrapper; kjanat/actionlint
is an actively maintained actionlint fork that ships its own action
(with built-in shellcheck), so use it directly.

In particular, the fork knows about the ubuntu-26.04 runner labels,
which upstream actionlint does not yet recognize.

Pin to the v1.17 tag rather than v1.17.0, as the former also pins
the action's docker image by digest.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin

Copy link
Copy Markdown
Collaborator Author

@jnovy ptal

@jnovy

jnovy commented Oct 29, 2026

Copy link
Copy Markdown
Collaborator

LGTM

@kolyshkin

Copy link
Copy Markdown
Collaborator Author

this should unblock #698

@jnovy jnovy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kolyshkin
kolyshkin merged commit 231a64a into containers:main Oct 29, 2026
35 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