-
Notifications
You must be signed in to change notification settings - Fork 72
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃殮(config) rename custom configuration files #23
Conversation
config/lms/utils.py
Outdated
# Load the content of a `settings.yaml` file placed in the current directory if any | ||
# This file is where customisable settings are stored for a given environment | ||
# Load the content of a `settings.yml` file placed in the current | ||
# directory if any This file is where customizable settings are stored |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing a dot
config/lms/utils.py
Outdated
# Load the content of a `credentials.vault.yaml` file placed in the current directory if | ||
# any. This file is where sensitive settings are stored for a given environment | ||
# Load the content of a `secrets.yml` file placed in the current | ||
# directory if any. This file is where sensitive settings are stored for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/settings/credentials
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor typos otherwise GTM.
Expected configurations files names to override settings are now: * `settings.yml` (prev. `settings.yaml`) as the `.yml` extension is more oftlenly used than `.yaml`. * `secrets.yml` (prev. `credentials.vault.yml`) as we try to make the generation of this file independent from ansible (vault).
fc33f63
to
2c4828d
Compare
This patch fixes a bug introduced by my latest PR (see #23). We expect a secrets.yml file.
This patch fixes a bug introduced by my latest PR (see #23). We expect a secrets.yml file.
This patch fixes a bug introduced by my latest PR (see #23). We expect a secrets.yml file.
Purpose
We need to make settings override via YAML files more generic.
Proposal
Expected configurations files names to override settings are now:
settings.yml
(prev.settings.yaml
) as the.yml
extension is moreoftlenly used than
.yaml
.secrets.yml
(prev.credentials.vault.yml
) as we try to makethe generation of this file independent from ansible (vault).
I've also slightly fixed docstrings/comments formatting (black style) 馃槈