Skip to content

feat: Support configuration via flags, env vars and config file#39

Open
irvinlim wants to merge 2 commits intomainfrom
irvinlim/feat/revamp-configuration
Open

feat: Support configuration via flags, env vars and config file#39
irvinlim wants to merge 2 commits intomainfrom
irvinlim/feat/revamp-configuration

Conversation

@irvinlim
Copy link
Owner

Revamps configuration handling for the application.

  • Supports configuration via environment variables (fixes Configure with environment variables #37) as well as configuration files (preferred way moving forward).
  • Revamp configuration handling to merge down configuration in the following order of precedence (from lowest to highest):
    • Configuration file (JSON and YAML both supported)
    • Environment variable
    • Command-line flags
  • Refactor existing command-line flags to unified entrypoint, and add notice to avoid adding too many superfluous command-line flags moving forward.
    • Currently I have not yet decided which flags (if any) shall be marked as deprecated, which should show a warning or error if we choose to fully deprecate them in the future, but may be considered for future plans to reduce maintenance burden.

@irvinlim irvinlim force-pushed the irvinlim/feat/revamp-configuration branch from c37cc53 to 185b860 Compare August 10, 2025 15:21
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.

Configure with environment variables

1 participant