Skip to content

refactor(settings): Normalize setttings to match pydantic #3

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 5, 2024

Conversation

heliocastro
Copy link
Collaborator

  • Refactored config naming to settings to match pydantic settings naming structure.
  • Enabled .env / .env.prod evaluation capability .
  • Add ORTHW_CONFIG_DIR and ORTHW_CONFIG_FILE variables that can be evaluated in runtime bypassing as environment file or inside .env. The variables are case sensitive.

@heliocastro heliocastro requested a review from tsteenbe July 5, 2024 08:58
@heliocastro heliocastro self-assigned this Jul 5, 2024
@heliocastro heliocastro force-pushed the heliocastro/config branch from 47e3691 to 46f8815 Compare July 5, 2024 08:59
- Refactored config naming to settings to match pydantic settings
naming structure.
- Enabled .env / .env.prod evaluation capability .
- Add ORTHW_CONFIG_DIR and ORTHW_CONFIG_FILE variables that can be
  evaluated in runtime bypassing as environment file or inside .env.
  The variables are case sensitive.

Signed-off-by: Helio Chissini de Castro <[email protected]>
@heliocastro heliocastro force-pushed the heliocastro/config branch from 46f8815 to 87be411 Compare July 5, 2024 09:06
@tsteenbe tsteenbe merged commit d2cf19f into main Jul 5, 2024
2 checks passed
@tsteenbe tsteenbe deleted the heliocastro/config branch July 5, 2024 09:14
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.

2 participants