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

[bug] Includes parameter not working #794

Closed
lbonini94 opened this issue Sep 2, 2022 · 1 comment · Fixed by #796
Closed

[bug] Includes parameter not working #794

lbonini94 opened this issue Sep 2, 2022 · 1 comment · Fixed by #796
Assignees
Labels
Docs Not a Bug Not a Problem, expected behavior

Comments

@lbonini94
Copy link

Describe the bug
The includes parameter is not loading the files properly

To Reproduce
Steps to reproduce the behavior:

  1. Having the following folder structure
├── conf
│   ├── database
│   │   └── postgre.toml
│   └── settings.toml
└── config.py
  1. Having the following config file
# config.py
from dynaconf import Dynaconf

settings = Dynaconf(
    envvar_prefix="DYNACONF",
    settings_files=['conf/settings.toml',
                    'conf/.secrets.toml'],
    includes=['conf/**/*.toml']
)
  1. Run dynaconf -i config.settings list and the parameters inside conf/database/postgre.toml are not showed.
Working in main environment 
A<int> 3
B<int> 2
  1. Having the folling .toml files
# conf/settings.toml
a=3
b=2
# conf/database/postgre.toml
included_host="0.0.0.0"

Expected behavior
Output:

Working in main environment 
A<int> 3
B<int> 2
INCLUDED_HOST<str> "0.0.0.0"

Environment (please complete the following information):

  • OS: OS: Pop!_OS 22.04 LTS x86_64
  • Dynaconf Version 3.1.9
@lbonini94 lbonini94 added the bug label Sep 2, 2022
rochacbruno added a commit that referenced this issue Sep 2, 2022
@rochacbruno rochacbruno added Not a Bug Not a Problem, expected behavior Docs and removed bug labels Sep 2, 2022
@rochacbruno
Copy link
Member

Not a bug, I added note to the docs and a test + explanation here: #796

@rochacbruno rochacbruno self-assigned this Sep 2, 2022
rochacbruno added a commit that referenced this issue Sep 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs Not a Bug Not a Problem, expected behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants