Skip to content

Add lint workflow running npm run lint per project - #99

Merged
cafedomancer merged 1 commit into
mainfrom
add-lint-workflow
Jun 17, 2026
Merged

Add lint workflow running npm run lint per project#99
cafedomancer merged 1 commit into
mainfrom
add-lint-workflow

Conversation

@cafedomancer

@cafedomancer cafedomancer commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Pull Request の作成時に GitHub Actions で自動的に Lint するようにした。

Summary by CodeRabbit

リリースノート

  • Chores
    • 自動コード検査機能を導入しました。プッシュおよびプルリクエスト時に複数プロジェクトのコード品質が自動的に検証されるようになり、コードの一貫性が向上します。

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@cafedomancer cafedomancer self-assigned this Jun 17, 2026
@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

main への push および pull_request をトリガとする GitHub Actions の lint ワークフローを新規追加した。01.fizzbuzz04.class の4プロジェクトをマトリクスで巡回し、Node.js 22 環境で npm cinpm run lint を実行する。

Changes

Lint CI ワークフロー

Layer / File(s) Summary
Lint ワークフロー定義
.github/workflows/lint.yml
main への push/pull_request をトリガに、01.fizzbuzz04.class の4プロジェクトをマトリクスで巡回し、actions/checkout@v5actions/setup-node@v5(Node.js 22、npm キャッシュ有り)でセットアップ後、npm cinpm run lint を順に実行するジョブを定義した。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐇 ぴょんぴょんとコードを跳ねる、
lint の番人、毎回チェック!
マトリクスでズラリ4列、
Node 22 が駆け抜ける 🚀
エラーは逃げられぬよ!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed プルリクエストのタイトルは、変更内容の主な目的である「各プロジェクトに対してnpm run lintを実行するLintワークフローの追加」を明確かつ簡潔に表現しており、変更内容と完全に対応しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch add-lint-workflow

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 and usage tips.

@cafedomancer
cafedomancer merged commit 79dfb63 into main Jun 17, 2026
4 of 5 checks passed
@cafedomancer
cafedomancer deleted the add-lint-workflow branch June 17, 2026 11:20
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