[CI検証] 静的解析ジョブ追加とキャッシュ残留フレーク修正 - #1
Conversation
- coupon-plugin ほか他プラグインと同じ構成に揃える - phpstan は objectManagerLoader がカーネルを起動し EccubeExtension が dtb_plugin を 読むため本体と DB が必要。DB 種別に依存しないので SQLite 1 構成で実行する - Entity のプロパティは ?type だが該当カラムは DB 上 not-null。Symfony のフォームは バリデーションの前にデータマッピングを行い空文字が null になるため非 nullable に できず、doctrine.columnType を ignoreErrors に入れる - php-cs-fixer が検出したコンストラクタ引数の整形を反映する - 配布パッケージには静的解析の設定を含めない Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
- Symfony の CacheClearCommand は REQUEST_TIME と filemtime の秒精度比較で コンテナを fresh とみなし、削除せずに正常終了することがある - eccube:plugin:enable が有効化前の boot でダンプしたコンテナ (プラグイン ルートなし) が同一秒内の cache:clear を素通りすると、cache:warmup は 既存ダンプを再利用するためルートが欠けたまま phpunit に到達する - cache:clear の判定に頼らず var/cache/test を直接削除する Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
29a8211 to
7f45d0b
Compare
|
検証完了（2 run 連続で全ジョブ緑）。upstream へ EC-CUBE#100 として提出したためクローズします。 |
1 participant
fork 上での CI 検証用 PR。upstream への提出前に pull_request イベントで全ジョブが緑になることを確認する。