ec-cube icon indicating copy to clipboard operation
ec-cube copied to clipboard

CustomizeディレクトリからSymfony Bundleをインストールできるよう修正

Open kurozumi opened this issue 2 years ago • 4 comments

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

kurozumi avatar Apr 15 '24 05:04 kurozumi

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.

codecov[bot] avatar Apr 17 '24 12:04 codecov[bot]

@kurozumi ありがとうございます! 不要な修正が混ざっているので、直していただいても良いでしょうか?

ji-eunsoo avatar May 15 '24 07:05 ji-eunsoo

@kurozumi ありがとうございます! よろしければ、テストコードの追加をお願いしたいです。

dotani1111 avatar May 20 '24 05:05 dotani1111

@dotani1111

テストを追加しました。

kurozumi avatar May 20 '24 07:05 kurozumi

@kurozumi ありがとうございます。 動作確認して問題なければ、マージしたいと思います。

dotani1111 avatar May 22 '24 08:05 dotani1111