Skip to content

chore: add actionlint to lint job#6

Closed
taro-28 wants to merge 1 commit into
mainfrom
chore/add-actionlint
Closed

chore: add actionlint to lint job#6
taro-28 wants to merge 1 commit into
mainfrom
chore/add-actionlint

Conversation

@taro-28
Copy link
Copy Markdown
Member

@taro-28 taro-28 commented May 27, 2026

Summary

  • .github/workflows/test.ymllint job に reviewdog/action-actionlint を追加し、action.yml と workflow YAML を CI で静的解析できるようにする。
  • 既存の ShellCheck(scripts/ 配下対象)と並列に動かす形にし、job 名を実態に合わせて ShellCheckLint に更新(他 job からは job ID lint で参照しているため互換)。
  • pin 形式は Pin third-party actions to full-length commit SHAs #5 で確立したルール(<40桁SHA> # <tag>)に従い reviewdog/action-actionlint@6fb7acc99f4a1008869fa8a0f09cfca740837d9d # v1.72.0
  • fail_level: error を明示してエラー時に CI を fail させる。

Verification

  • ローカルで actionlint -color を実行し、既存の action.yml / .github/workflows/*.yml が clean で通ることを確認済み(exit 0、出力なし)。
  • ローカルで shellcheck scripts/*.sh も exit 0。
  • CI 上でも新規 actionlint ステップが pass することを確認する。

Test plan

  • Test workflow が成功し、Lint job 内で actionlint ステップが追加されていること
  • 既存の ShellCheck ステップが引き続き pass すること
  • その他 job(bats, integration, cli-install)が壊れていないこと

🤖 Generated with Claude Code

Run reviewdog/action-actionlint alongside ShellCheck so that
`action.yml` and workflow YAML get static analysis (syntax, expression
evaluation, mismatched inputs) on every push and PR. Pinned to a
40-char commit SHA per the policy established in #5.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

mock sync output

@github-actions
Copy link
Copy Markdown

mock sync output

@github-actions
Copy link
Copy Markdown

mock sync output

@github-actions
Copy link
Copy Markdown

mock sync output

@taro-28 taro-28 mentioned this pull request May 27, 2026
3 tasks
@taro-28
Copy link
Copy Markdown
Member Author

taro-28 commented May 27, 2026

PR #7 (zizmor) で SHA pin 強制を CI で行う方針に切り替えるため close。actionlint は本タスクのスコープ(pin 強制)と別軸であり、ユーザー判断で当面は導入を見送り。

@taro-28 taro-28 closed this May 27, 2026
@taro-28 taro-28 deleted the chore/add-actionlint branch May 27, 2026 06:56
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