Skip to content

ci: replace actionlint with jactionlint - #7697

Merged
viceice merged 2 commits into
mainfrom
viceice/ci/jactionlint
Oct 9, 2026
Merged

viceice merged 2 commits into
mainfrom
viceice/ci/jactionlint

Conversation

@viceice

@viceice viceice commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

Changes

  • Replaces actionlint with jactionlint, its actively maintained fork, in the lint-actions job, using its official image ghcr.io/jdx/jactionlint pinned by digest.
  • Adds jactionlint to mise.toml, so mise install makes it available locally, and mentions it in AGENTS.md.

Context

Please select one of the following:

  • This closes an existing Issue, Closes: #
  • This doesn't close an Issue, but I accept the risk that this PR may be closed if maintainers disagree with its opening or implementation

AI assistance disclosure

Did you use AI tools to create any part of this pull request?

Please select one option and, if yes, briefly describe how AI was used (e.g., code, tests, docs) and which tool(s) you used.

  • No — I did not use AI for this contribution.
  • Yes — minimal assistance (e.g., IDE autocomplete, small code completions, grammar fixes).
  • Yes — substantive assistance (AI-generated non‑trivial portions of code, tests, or documentation).
  • Yes — other (please describe):

Claude Opus 5.5 (Claude Code) made the switch and ran jactionlint locally, both from the image and through mise.

Use of AI in replying to PR comments

Who answers review comments:

  • @viceice will read and reply directly. Name the account.
  • An agent will draft replies and @username will read them before they are posted. Name the account.
  • An agent will draft replies and reply autonomously. This is heavily discouraged, and we prefer that there are humans in the loop
  • Nobody has explicitly committed to replying.

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified tests

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <michael.kriese+claude-code@mend.io>
@coderabbitai

coderabbitai Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 23 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: containerbase/coderabbit/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: bf46ec8b-8944-4720-b7cd-0b459227cc6c

📥 Commits

Reviewing files that changed from the base of the PR and between d1a4211 and 9d26062.


📒 Files selected for processing (3)
  • .github/workflows/build.yml
  • AGENTS.md
  • mise.toml


  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

ci: replace actionlint with jactionlint in CI and local tooling

⚙️ Configuration changes 📝 Documentation 🕐 10-20 Minutes

Grey Divider

AI Description

• Replace actionlint with digest-pinned jactionlint for GitHub workflow and local action checks.
• Pin the same jactionlint version for local use through mise and document its command.
Diagram

graph TD
  CI["CI lint job"] --> J["jactionlint"] --> F["Workflows and actions"]
  M["mise local CLI"] --> J
Loading
High-Level Assessment

Keep the existing lint job and replace its pinned image, with mise supplying the same version locally. A separate binary installation in CI would add setup without a clear benefit for this contained tool swap.

Files changed (3) +4 / -2

Documentation (1) +1 / -0
AGENTS.mdDocument the local jactionlint command +1/-0

Document the local jactionlint command

• Adds 'mise exec -- jactionlint' to the documented commands for checking GitHub workflows and local actions.

AGENTS.md

Other (2) +3 / -2
build.ymlRun jactionlint in the CI lint-actions job +2/-2

Run jactionlint in the CI lint-actions job

• Replaces the actionlint Docker image with the digest-pinned jactionlint 1.8.2 image. The existing lint arguments and sparse checkout remain in place, and the checkout comment names the new tool.

.github/workflows/build.yml

mise.tomlPin jactionlint for local use +1/-0

Pin jactionlint for local use

• Adds jactionlint 1.8.2 to mise, matching the version used by the CI image.

mise.toml

@qodo-code-review

qodo-code-review Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Tip of the day
💡 Did you know, you can show, collapse, or hide each part of a finding: code, evidence, and all

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Grey Divider

Qodo Logo

Co-Authored-By: Claude Opus 5.5 <michael.kriese+claude-code@mend.io>
@viceice
viceice enabled auto-merge October 9, 2026 09:49
@viceice
viceice added this pull request to the merge queue Oct 9, 2026
Merged via the queue into main with commit 8bd6822 Oct 9, 2026
58 checks passed
@viceice
viceice deleted the viceice/ci/jactionlint branch October 9, 2026 10:44
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