Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ComposerUpdateによるSymfonyの仕様変更への対応 #4163

Merged
merged 4 commits into from
May 15, 2019
Merged

ComposerUpdateによるSymfonyの仕様変更への対応 #4163

merged 4 commits into from
May 15, 2019

Conversation

kazumiiiiiiiiiii
Copy link
Contributor

以下を参考にコメントを作成してください。

概要(Overview・Refs Issue)

ComposerUpdateにより、以下のプルリクの取り込みが必要となったため追加。

#2865

方針(Policy)

3.1に取り込まれた修正内容を参考にマージ。

実装に関する補足(Appendix)

なし

テスト（Test)

Codeceptionを実施。以下、codeceptionの修正PullRequest

EC-CUBE/eccube-codeception#157

相談（Discussion）

なし

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

  • マイナーバージョンでは、機能・プラグイン・デザインテンプレート互換性を損なう変更は原則取り込みません。
  • 既存機能の仕様変更
  • フックポイントの呼び出しタイミングの変更
  • フックポイントのパラメータの削除・データ型の変更
  • twigファイルに渡しているパラメータの削除・データ型の変更
  • Serviceクラスの公開関数の、引数の削除・データ型の変更
  • 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

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

@kiy0taka kiy0taka merged commit ccd3bc0 into EC-CUBE:3.0 May 15, 2019
@chihiro-adachi chihiro-adachi added this to the 3.0.18 milestone May 22, 2019
@kazumiiiiiiiiiii kazumiiiiiiiiiii deleted the 3.0.18_version branch May 28, 2019 07:54
@chihiro-adachi chihiro-adachi mentioned this pull request Jun 14, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug:Critical
Projects
None yet
Milestone
3.0.18
Development

Successfully merging this pull request may close these issues.
4 participants
@kazumiiiiiiiiiii @kiy0taka @chihiro-adachi @nanasess