Skip to content

[4.1] Apply composer update "symfony/*"#4813

Merged
okazy merged 1 commit intoEC-CUBE:4.1from
nanasess:composer-update-20201221
Dec 22, 2020
Merged

[4.1] Apply composer update "symfony/*"#4813
okazy merged 1 commit intoEC-CUBE:4.1from
nanasess:composer-update-20201221

Conversation

@nanasess
Copy link
Copy Markdown
Contributor

@nanasess nanasess commented Dec 21, 2020

概要(Overview・Refs Issue)

  • Symfony 4.4.18 へアップデート

方針(Policy)

以下のコマンドを実行

composer update "symfony/*"

実装に関する補足(Appendix)

テスト(Test)

ユニットテストが通るのを確認

相談(Discussion)

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

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

レビュワー確認項目

  • 動作確認
  • コードレビュー
  • E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
  • 互換性が保持されているか
  • セキュリティ上の問題がないか

nanasess added a commit to nanasess/ec-cube that referenced this pull request Dec 22, 2020
@okazy okazy added dependencies Pull requests that update a dependency file improvement 機能改善 Status: ready-for-merge labels Dec 22, 2020
@okazy okazy added this to the 4.1 milestone Dec 22, 2020
@okazy okazy merged commit fa8aaeb into EC-CUBE:4.1 Dec 22, 2020
@okazy
Copy link
Copy Markdown
Contributor

okazy commented Dec 22, 2020

ありがとうございます!マージしました。

@nanasess nanasess deleted the composer-update-20201221 branch October 3, 2022 07:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file improvement 機能改善 Status: ready-for-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants