Skip to content

feat(ci): 4.3ブランチのCI/CD改善を4.3-symfony7に取り込む#6689

Open
dotani1111 wants to merge 6 commits intoEC-CUBE:4.3-symfony7from
dotani1111:feat/symfony7-ci-sync-from-4.3
Open

feat(ci): 4.3ブランチのCI/CD改善を4.3-symfony7に取り込む#6689
dotani1111 wants to merge 6 commits intoEC-CUBE:4.3-symfony7from
dotani1111:feat/symfony7-ci-sync-from-4.3

Conversation

@dotani1111
Copy link
Copy Markdown
Contributor

@dotani1111 dotani1111 commented Mar 26, 2026

概要(Overview・Refs Issue)

4.3ブランチのCI/CDの修正を取り込みます。
Symfony7対応として、PHP 8.2-8.5、ECCUBE_AUTH_MAGIC、register_argc_argv for PHP 8.5は維持しています。

方針(Policy)

以下の変更が含まれています。

  • actions/cacheをv5ピン固定SHAに更新
  • main.yml: permissions/concurrency追加、rector追加、deny-test有効化、coverageをunit-testのみに依存変更
  • rector.yml: 新規作成(4.3から移植)
  • phpstan.yml/php-cs-fixer.yml: permissions追加、shivammathurピン固定、キャッシュステップ追加
  • unit-test.yml: shivammathurピン固定、DBスキーマキャッシュ追加、nick-invision/retryでラップ
  • e2e-test.yml: グループ分割(admin01-a/b等)、shivammathurピン固定、npm/フォントキャッシュ、retryラップ、upload-artifact v7ピン固定
  • e2e-test-throttling.yml: shivammathurピン固定、awalsh128フォントキャッシュ、upload-artifact v7ピン固定
  • plugin-test.yml: shivammathurピン固定、awalsh128でpostgresql-clientキャッシュ、upload-artifact v7ピン固定
  • coverage.yml: shivammathurピン固定、pcovインストールをマトリクス変数化
  • deny-test.yml: shivammathurピン固定、ECCUBE_AUTH_MAGICを.envに追記

実装に関する補足(Appendix)

テスト(Test)

相談(Discussion)

マイナーバージョン互換性保持のための制限事項チェックリスト

  • 既存機能の仕様変更はありません
  • フックポイントの呼び出しタイミングの変更はありません
  • フックポイントのパラメータの削除・データ型の変更はありません
  • twigファイルに渡しているパラメータの削除・データ型の変更はありません
  • Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
  • 入出力ファイル(CSVなど)のフォーマット変更はありません

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか
    • 権限を超えた操作が可能にならないか
    • 不要なファイルアップロードがないか
    • 外部へ公開されるファイルや機能の追加ではないか
    • テンプレートでのエスケープ漏れがないか

- actions/cacheをv5ピン固定SHAに更新
- main.yml: permissions/concurrency追加、rector追加、deny-test有効化、coverageをunit-testのみに依存変更
- rector.yml: 新規作成(4.3から移植)
- phpstan.yml/php-cs-fixer.yml: permissions追加、shivammathurピン固定、キャッシュステップ追加
- unit-test.yml: shivammathurピン固定、DBスキーマキャッシュ追加、nick-invision/retryでラップ
- e2e-test.yml: グループ分割(admin01-a/b等)、shivammathurピン固定、npm/フォントキャッシュ、retryラップ、upload-artifact v7ピン固定
- e2e-test-throttling.yml: shivammathurピン固定、awalsh128フォントキャッシュ、upload-artifact v7ピン固定
- plugin-test.yml: shivammathurピン固定、awalsh128でpostgresql-clientキャッシュ、upload-artifact v7ピン固定
- coverage.yml: shivammathurピン固定、pcovインストールをマトリクス変数化
- deny-test.yml: shivammathurピン固定、ECCUBE_AUTH_MAGICを.envに追記

PHP 8.2-8.5、ECCUBE_AUTH_MAGIC、register_argc_argv for PHP 8.5は維持

Made-with: Cursor
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9f60c8de-22c0-4828-88a7-8c8eb2e283a6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@dotani1111 dotani1111 marked this pull request as draft April 1, 2026 06:55
@dotani1111 dotani1111 marked this pull request as ready for review April 7, 2026 08:36
- nick-invision/retryを廃止し、失敗時のみ1回リトライするシェルに変更(成功時は2回目を実行しない)

- PHPUnit 12向けに--exclude-groupをカンマ区切りから複数指定へ変更

- リトライ前にdoctrine:database:drop等でDBを初期セットアップと同様に再構築

Made-with: Cursor
- PHPUnit/Codeceptionいずれもextensionsにpcovを追加

- sury/PPAのphpX.Y-pcovとsetup-phpのPHP版がずれてdebが404になる問題を解消

Made-with: Cursor
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.68%. Comparing base (29c1c4c) to head (998d0f1).
⚠️ Report is 1 commits behind head on 4.3-symfony7.

Additional details and impacted files
@@               Coverage Diff                @@
##           4.3-symfony7    #6689      +/-   ##
================================================
+ Coverage         82.28%   82.68%   +0.40%     
================================================
  Files               482      482              
  Lines             25711    25765      +54     
================================================
+ Hits              21156    21304     +148     
+ Misses             4555     4461      -94     
Flag Coverage Δ
E2E 82.68% <ø> (+0.40%) ⬆️
Unit 82.68% <ø> (+0.40%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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