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

all: permit to read config from yaml not from env #256

Merged
merged 1 commit into from
Aug 12, 2022
Merged

all: permit to read config from yaml not from env #256

merged 1 commit into from
Aug 12, 2022

Conversation

yottahmd
Copy link
Collaborator

@yottahmd yottahmd commented Aug 12, 2022

This PR closes #248

Breaking Change

This change disables env config below:

  • DAGU__DATA
  • DAGU__LOGS
  • DAGU__SUSPEND_FLAGS_DIR
  • DAGU__ADMIN_CONFIG
  • DAGU__ADMIN_PORT
  • DAGU__ADMIN_NAVBAR_COLOR
  • DAGU__ADMIN_NAVBAR_TITLE
  • DAGU__ADMIN_LOGS_DIR
  • DAGU__ADMIN_DAGS_DIR

The user can still set the group of values below within the global config file:

  • DAGU__ADMIN_LOGS_DIR
  • DAGU__ADMIN_PORT
  • DAGU__ADMIN_NAVBAR_COLOR
  • DAGU__ADMIN_NAVBAR_TITLE

example:

logs: /var/dagu/
port: 8080
navbarColor: red
navbarTitle: production

@yottahmd yottahmd merged commit 0958a7f into main Aug 12, 2022
@yottahmd yottahmd deleted the develop branch August 12, 2022 11:30
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.

Move environ config to global config
1 participant