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.

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

[4.1-beta2]マージ漏れ修正と docker build のテストを追加 #5009

Merged
merged 2 commits into from
Apr 12, 2021

Conversation

nanasess
Copy link
Contributor

@nanasess nanasess commented Apr 9, 2021

概要(Overview・Refs Issue)

  • hirak/prestissimo は不要
  • composer2だとprestissimoはエラーになる #4810
  • 新たに docker build のテストを追加
    • docker 関連のファイルが修正された場合のみ実行する
    • docker-compose 経由で E2Eテスト(admin01) を実行し、動作確認

テスト(Test)

  • docker build のテストを追加

相談(Discussion)

admin01 以外の E2E テストはエラーになってしまい、テストケースの修正が必要なのでスキップ

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

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

レビュワー確認項目

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

@nanasess nanasess changed the title [4.1-beta2]マージ漏れ修正 [4.1-beta2]マージ漏れ修正と docker build のテストを追加 Apr 9, 2021
@nanasess nanasess force-pushed the remove-prestissimo branch from 770969b to 861e694 Compare April 9, 2021 06:55
@nanasess
Copy link
Contributor Author

nanasess commented Apr 9, 2021

docker build のテストを追加してみました。良さげなら4.0にもバックポートしたいと思います

php-version: ${{ matrix.php }}

- name: composer install
run: composer install --dev --no-interaction -o --apcu-autoloader
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

こちらの composer install は必要ですか?
docker-compose build で Docker 内のライブラリはインストール済みかと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codeception はローカル側で動かしているので、ローカル側にインストールが必要なんです😥

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

そうでした😌
ご確認ありがとうございます。

@okazy okazy added the improvement 機能改善 label Apr 12, 2021
@okazy okazy added this to the 4.1 milestone Apr 12, 2021
Copy link
Contributor

@okazy okazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

workflowの内容とテストが問題なく動作していることをを確認しました。

@okazy okazy merged commit 5e9fd61 into EC-CUBE:4.1-beta2 Apr 12, 2021
@okazy
Copy link
Contributor

okazy commented Apr 12, 2021

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

@nanasess nanasess deleted the remove-prestissimo branch October 3, 2022 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants