-
Notifications
You must be signed in to change notification settings - Fork 8
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
Update config to match the latest conventions #929
Comments
Have some SLA time this week going to pick this up and bundle it into another PR if everyone is okay with that? |
Also a candidate for @pjdiiori to pick up with his SLA shadow hours |
Is this just as simple as renaming "releases.exs" to "runtime.exs"? I've found some examples that place all the configurations within a |
I just realized something about this:
So we have to be careful here. We probably have some compile-time config for test and dev that we don't want to overwrite with environment variables even though we do want to use environment for those configs in prod. Some resources: |
Recent elixir versions use a config scheme that's easier to understand, with a clear separation between compile-time and runtime configutation: https://hexdocs.pm/mix/Mix.Tasks.Release.html#module-runtime-configuration
We should update the template to match this setup so onboarding engineers can find things where they expect and can rely on the latest elixir docs for guidance.
The text was updated successfully, but these errors were encountered: