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

Allow system, global and local config, similar to how git configs are loaded #105

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JonTheNiceGuy
Copy link

@JonTheNiceGuy JonTheNiceGuy commented Jan 19, 2021

Allow system, global and local config, similar to git configs

Viable config file paths are:

  • System
    • /etc/default/adr-config
    • /etc/adr-config
  • Global (user)
    • ~/.adr-config
  • Local (repo)
    • $(_adr_dir)/.adr-config

This change relies on _adr_dir not trying to load config, so
it also incorporates this removal, even though this may have been
removed in another branch/PR.

Tests are included. If it's not possible to run sudo based tests in
the CI/CD system, then these should be removed.

Viable config file paths are:
* System
  * `/etc/default/adr-config`
  * `/etc/adr-config`
* Global (user)
  * `~/.adr-config`
* Local (repo)
  * `$(_adr_dir)/.adr-config`

This change relies on `_adr_dir` not trying to load config, so
it also incorporates this removal, even though this may have been
removed in another branch/PR.

Tests are included. If it's not possible to run sudo based tests in
the CI/CD system, then these should be removed.
halostatue pushed a commit to halostatue/adr-tools that referenced this pull request Jul 20, 2024
Viable config file paths are:
* System
  * `/etc/default/adr-config`
  * `/etc/adr-config`
* Global (user)
  * `~/.adr-config`
* Local (repo)
  * `$(_adr_dir)/.adr-config`

This change relies on `_adr_dir` not trying to load config, so
it also incorporates this removal, even though this may have been
removed in another branch/PR.

Tests are included. If it's not possible to run sudo based tests in
the CI/CD system, then these should be removed.

npryce#105
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.

1 participant