ci: pin actions/checkout and shivammathur/setup-php action to specific commit#33
Conversation
WalkthroughGitHub Actions の 2 つのワークフロー(cron.yml, psalm.yml)で、 Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
✨ Finishing Touches🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
.github/workflows/cron.yml (1)
23-23: actions/checkout を公式リリースタグSHAに固定検討
- 現在の
actions/checkout@ff7abcd0c3c05ccf6adc123a8cd1fd4fb30fb493は Node.js 24 サポート要件を追加した upstream コミットですが、将来の main ブランチ更新の影響を受けないよう、以下いずれかのリリースタグSHAへ置き換えを推奨します:
- v5:
08c6903cd8c0fde910a37f88322edcfb5dd907a8- v4.2.2:
11bd71901bbe5b1630ceea73d27597364c9af683- Dependabot でワークフローファイルの Actions ピン更新を監視する運用もご検討ください。
.github/workflows/psalm.yml (1)
31-31: checkout の SHA 固定は妥当。より安定させるならリリースタグのコミット SHA へ
ff7abcd…は Node.js 24 要件の README 更新コミットです。自前ランナー併用時は Runner v2.327.1 以上を満たすことを確認してください。長期の安定運用には、v4(現行利用互換)または v5 の「リリースコミット SHA」に固定する方針を推奨します。(github.com)上記(cron と同様)のスクリプトで該当タグのコミット SHA を取得し、置き換え可否をご確認ください。
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (2)
.github/workflows/cron.yml(1 hunks).github/workflows/psalm.yml(1 hunks)
🔇 Additional comments (2)
.github/workflows/cron.yml (1)
18-18: SHA ピン留め OK(setup-php 2.35.4 のコミットに一致)
ec406be…は upstream の「Bump version to 2.35.4」コミットで正当性を確認できました。PHP 8.4 は安定版としてサポートされています。(github.com).github/workflows/psalm.yml (1)
25-25: SHA ピン留め OK(setup-php 2.35.4 のコミットに一致)
ec406be…は upstream の 2.35.4 コミットで正しく固定されています。PHP 8.4 サポートも適合。(github.com)
セキュリティ強化のために actions/checkout と shivammathur/setup-php のバージョンを特定のコミット SHA に固定しました。
Summary by CodeRabbit