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.

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

cs-fixerを適用 #4044

Merged

chihiro-adachi merged 1 commit into EC-CUBE:4.0 from okazy:fix-with-cs-fixer

Jan 20, 2019

Contributor

okazy commented Dec 12, 2018

概要(Overview・Refs Issue)

cs-fixerを適用

方針(Policy)

マイグレーションファイルも含めて全ファイルにfixerを適用

実装に関する補足(Appendix)

目視で処理が変わるような変更がないことを確認

テスト（Test)

ローカルでフロント画面のTOPと管理画面のホームがエラーなく表示できることを確認

相談（Discussion）

なし

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

なし

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


          fix cs-fixer

37cd072

okazy added this to the 4.0.2 milestone

hiro-t59 added the Status: needs-review label

chihiro-adachi added Status: ready-for-merge and removed Status: needs-review labels

chihiro-adachi approved these changes

View reviewed changes

chihiro-adachi merged commit ad03d2d into EC-CUBE:4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment