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

Use mailcatcher appveyor #258

Merged

Conversation

nanasess
Copy link
Contributor

概要(Overview・Refs Issue)

方針(Policy)

  • VS2017 のイメージで Docker コンテナが利用できるようになったので、 MailCatcher の Docker コンテナを追加

テスト(Test)

  • メール関係のテストがスキップされずに通ることを確認

相談(Discussion)

  • Windows Server Core のイメージは6GBくらいあり、 pull に時間がかかる(5分程度)
  • 本来は Nano Server を使用したいが、以下の問題があり、現在は Windows Server Core のイメージを使用している
    • Ruby 2.3.x までは SQLIte3 の gem のビルドに失敗する
    • Ruby 2.4.x 以降は Nano Server で Ruby が動かない

@coveralls
Copy link

Coverage Status

Coverage increased (+1.06%) to 23.7% when pulling 0d61827 on nanasess:use-mailcatcher-appveyor into 66eed5d on EC-CUBE:improve/php7.

@chihiro-adachi chihiro-adachi added this to the 2.17.0 milestone Apr 23, 2019
@chihiro-adachi chihiro-adachi merged commit 4f13920 into EC-CUBE:improve/php7 Apr 26, 2019
@chihiro-adachi
Copy link
Contributor

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

@nanasess nanasess deleted the use-mailcatcher-appveyor branch May 23, 2019 03:00
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.

4 participants