Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Setup default localhost db config #1322

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

Merged
merged 1 commit into from
Jul 13, 2021
Merged

Setup default localhost db config #1322

merged 1 commit into from
Jul 13, 2021

Conversation

nanophate
Copy link
Member

@nanophate nanophate commented Jul 13, 2021

背景

ENV.fetch に書かれていた値は対象の環境変数がないときのデフォルト値です。その何も指定しないことにしたのも昔の僕のようですが、デフォルト値を設定しないことにメリットはないので、環境の違いは環境変数で切り替えるようにすればいいかと思いました。エスパーすると、ローカルサーバー とおっしゃっているので、何も環境変数を設定していない結果、ただデフォルトの localhost につなぎにいって失敗されているのでしょう。docker を基準に環境は数年前に作ったのでデフォルト値もそれを基準としています。

cf. #1317 (comment)

やったこと

default の設定はあったほうが良さそうなので、設定しました。docker の場合は env.sample で指定があるので、ローカルで利用しやすいように、何も設定して無いローカルのdbを基準に設定しました。

確認方法

bin/setup を実行できる

@nanophate nanophate requested a review from Yuppymam July 13, 2021 02:30
Copy link
Contributor

@Yuppymam Yuppymam left a comment

Choose a reason for hiding this comment

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

環境変数をセットして、ローカルでbin/setupが動くことを確認しました👌

@yasulab
Copy link
Member

yasulab commented Jul 13, 2021

PR マージして、自分の方でもセットアップ試してみますね 🛠💨

@yasulab yasulab merged commit d433a4d into master Jul 13, 2021
@yasulab yasulab deleted the add-default-env branch July 13, 2021 03:03
@yasulab
Copy link
Member

yasulab commented Jul 13, 2021

マージしてしまった直後でメッチャ恐縮ですが、Railsの環境変数周りを更新するときはタイミングを合わせて更新したいので、いったんマージ前の状態 (#1322 マージ後の状態) に戻させていただきました 😂😂😂 cc/ @nanophate @Yuppymam 897b68d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants