-
Notifications
You must be signed in to change notification settings - Fork 11
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
🔧 added first draft of release files #30
Conversation
e99b4be
to
2a9b9ee
Compare
config/prod.exs
Outdated
@@ -51,4 +51,4 @@ config :logger, level: :info | |||
|
|||
# Finally import the config/prod.secret.exs which loads secrets | |||
# and configuration from environment variables. | |||
import_config "prod.secret.exs" | |||
# import_config "prod.secret.exs" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have you tried running the local phoenix server in prod env, i.e. MIX_ENV=prod mix phx.server
.
This failed when I tried executing this locally because database credentials were missing. We should enable this.
@@ -2,18 +2,42 @@ | |||
# from environment variables. You can also hardcode secrets, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As mentioned in previous remark, we should also define the ENV variables in config/prod.exs
.
ref #27