Open
Description
The load_local_env_vars module seems unnecessary and is weird that we call that function in prod (it's a no-op). It creates a situation where local code branch is different from prod code based on environment variables.
One idea
If we rename local.env
to dotenv then it will automatically load.
Pros:
- This will also help remove code branches that differ between local and prod systems.
- This will allow local developers to have their personal config changes without accidentally committing it and messing up other people's configs. Shared configs can still go in something like
.env.template
.
Cons:
- The con is that new config changes will need to be communicated to devs so they can pull new config changes from .env.template.
More context
Also see this PR discussion https://github.com/navapbc/template-application-flask/pull/138/files#r1105922982