Skip to content
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

Support multiple envs for setting #397

Closed
wants to merge 2 commits into from

Conversation

kairoaraujo
Copy link
Member

  • The old repository.refresh_settings handling the IServices
    configuration was moved to the interfaces
  • Now the way to importa a I{Services} uses directly the dynaconf
  • Adds better error mensages for wrong configuration
  • Support a service has one setting with multiple environment variables
    what is helpful for third-part libraries that uses environment vars

Kairo de Araujo added 2 commits September 20, 2023 14:26
- The old `repository.refresh_settings` handling the IServices
  configuration was moved to the `interfaces`
- Now the way to importa a `I{Services}` uses directly the dynaconf
- Adds better error mensages for wrong configuration
- Support a service has one setting with multiple environment variables
  what is helpful for third-part libraries that uses environment vars

Signed-off-by: Kairo de Araujo <[email protected]>
- Improve the comments
- Implement a better test for the all settings None
- Add asserts related to the stub

Signed-off-by: Kairo de Araujo <[email protected]>
@MVrachev
Copy link
Member

Superceeds: #389.

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