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

Docker環境整備 #347

Merged
merged 5 commits into from Jul 5, 2018

Conversation

Projects
None yet
2 participants
@shishi
Copy link
Contributor

shishi commented Jul 4, 2018

関連Issue

#346

やったこと

  • dotenv の導入
    • dockerの設定及びテストで環境変数を多用するため
  • database.ymlをdocker使用時でもそうでなくても動作するよう書き換え
  • Dockerfile, docker-compose, .dockerignore の追加
    • rails, postgresql, data volume のイメージ構成
  • README へDockerでの環境構築について追記

気になったこと

Scrivitoが必要とする環境変数のうち、"SCRIVITO_EMAIL"と"SCRIVITO_PASSWORD"は特定のものと定まっていないということなので環境変数にする必要はなく、テストにふさわしい文字列をテスト中で設定しておけば良いのではと思いました。

shishi added some commits Jul 4, 2018

@yasulab

This comment has been minimized.

Copy link
Member

yasulab commented Jul 5, 2018

README も追記していただきありがとうございます! 素敵ですね 😻

Scrivitoが必要とする環境変数のうち、"SCRIVITO_EMAIL"と"SCRIVITO_PASSWORD"は特定のものと定まっていないということなので環境変数にする必要はなく、テストにふさわしい文字列をテスト中で設定しておけば良いのではと思いました。

こちら確かに...!! とりあえず Issue にまとめておきました! 📝 💨 #348

@yasulab

This comment has been minimized.

Copy link
Member

yasulab commented Jul 5, 2018

良さそうなので一旦マージしますね! PRありがとうございました! 😸

@yasulab yasulab merged commit 54306b2 into coderdojo-japan:master Jul 5, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@shishi shishi deleted the shishi:docker branch Jul 5, 2018

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