Skip to content

CI: switch to kjanat/actionlint v1.17.0 - #259

Open
kolyshkin wants to merge 1 commit into
moby:mainfrom
kolyshkin:actionlint-kjanat-v1.16.0
Open

kolyshkin wants to merge 1 commit into
moby:mainfrom
kolyshkin:actionlint-kjanat-v1.16.0

Conversation

@kolyshkin

@kolyshkin kolyshkin commented Sep 9, 2026 •

Copy link
Copy Markdown
Collaborator

The devops-actions/actionlint wrapper runs an old rhysd/actionlint
release. Use the maintained kjanat/actionlint fork's own action instead,
pinned to the commit that pins its container image by digest (as
recommended by its README; the release tag itself references the image
by a floating tag).

Since v1.17.0 knows about the ubuntu-26.04 runner label, the
self-hosted-runner escape hatch in .github/actionlint.yaml is no
longer needed, so drop it.

Checked locally with the v1.17.0 binary (the same one the action's
container runs), without .github/actionlint.yaml: no findings.

@kolyshkin
kolyshkin force-pushed the actionlint-kjanat-v1.16.0 branch from 1ee9fac to 0a03165 Compare September 9, 2026 17:50
The devops-actions/actionlint wrapper runs an old rhysd/actionlint
release. Use the maintained kjanat/actionlint fork's own action
instead, pinned to the commit that pins its image by digest.

Since v1.17.0 knows about the ubuntu-26.04 runner label, the
self-hosted-runner escape hatch in .github/actionlint.yaml is no
longer needed, so drop it.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin
kolyshkin force-pushed the actionlint-kjanat-v1.16.0 branch from 0a03165 to 273083b Compare October 5, 2026 06:28
@kolyshkin kolyshkin changed the title CI: switch to kjanat/actionlint v1.16.0 CI: switch to kjanat/actionlint v1.17.0 Oct 5, 2026
@kolyshkin
kolyshkin requested review from thaJeztah and vvoland October 5, 2026 06:31
with:
persist-credentials: false
- uses: devops-actions/actionlint@ec02b36684b2f574f1d219ad0a43b082e46bf3e4 # v0.1.13
- uses: kjanat/actionlint@722799fa4b8cc6734debb97bae24aca9e0dbaf59 # v1.17

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Perhaps this should use v1.17.0 (instead of v1.17, which may be a rolling tag?)

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