CustomizeディレクトリからSymfony Bundleをインストールできるよう修正
概要(Overview・Refs Issue)
CustomizeディレクトリからSymfony BundleをインストールできるようKernel.phpを修正しました。
方針(Policy)
実装に関する補足(Appendix)
テスト(Test)
以下のコードでCustomizeディレクトリのBundleが読み込まれるのを確認しました。
<?php
namespace Customize\Bundle;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\HttpKernel\Bundle\Bundle;
class CustomizeBundle extends Bundle
{
public function build(ContainerBuilder $contaner): void
{
parent::build($contaner);
var_dump('hello world.');
}
}
app/Customize/Resource/config/bundles.php
<?php
return [
\Customize\Bundle\CustomizeBundle::class => ['all' => true]
];
相談(Discussion)
マイナーバージョン互換性保持のための制限事項チェックリスト
- [ ] 既存機能の仕様変更はありません
- [ ] フックポイントの呼び出しタイミングの変更はありません
- [ ] フックポイントのパラメータの削除・データ型の変更はありません
- [ ] twigファイルに渡しているパラメータの削除・データ型の変更はありません
- [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
- [ ] 入出力ファイル(CSVなど)のフォーマット変更はありません
レビュワー確認項目
- [ ] 動作確認
- [ ] コードレビュー
- [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
- [ ] 互換性が保持されているか
- [ ] セキュリティ上の問題がないか
- [ ] 権限を超えた操作が可能にならないか
- [ ] 不要なファイルアップロードがないか
- [ ] 外部へ公開されるファイルや機能の追加ではないか
- [ ] テンプレートでのエスケープ漏れがないか
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 82.80%. Comparing base (
4db7ae7) to head (6350e49).
Additional details and impacted files
@@ Coverage Diff @@
## 4.3 #6141 +/- ##
=======================================
Coverage 82.80% 82.80%
=======================================
Files 480 480
Lines 26431 26437 +6
=======================================
+ Hits 21885 21891 +6
Misses 4546 4546
| Flag | Coverage Δ | |
|---|---|---|
| E2E | 82.80% <100.00%> (+<0.01%) |
:arrow_up: |
| Unit | 82.80% <100.00%> (+<0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@kurozumi ありがとうございます! 不要な修正が混ざっているので、直していただいても良いでしょうか?
@kurozumi ありがとうございます! よろしければ、テストコードの追加をお願いしたいです。
@dotani1111
テストを追加しました。
@kurozumi ありがとうございます。 動作確認して問題なければ、マージしたいと思います。