Skip to content

Remove redundant ways to load environment variables #60

Description

@dennyabrain

Which feature or component needs enhancement?
The way we manage environment variables in the project

Describe the current behavior
We seem to have two redundant approaches to loading environment variables from the *.env files.

def get_settings() -> Settings:

and
def load_environment():

Describe the enhancement you'd like
The settings variable exposed in config.py seems to be the one we are relying heavily on throughout the codebase. So we can remove the load_env.py and all calling code referencing that file.

Why is this enhancement needed?
This will prevent confusing overrides of environment variables due to the two approaches interacting in unexpected manner.

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions