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.

Sign up for GitHub

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

Jump to bottom

update config handling: #15

Open
gernotstarke opened this issue Nov 18, 2024 · 2 comments
Open

update config handling: #15

gernotstarke opened this issue Nov 18, 2024 · 2 comments
Assignees
@gernotstarke
Labels
codeClimate coding stuff enhancement New feature or request

Comments

@gernotstarke
Copy link
Member

the MVP skipped several config items (header, footer, chapter-ids) for sake of faster speed-to-market.

Those should be added now.

The former config package should be (re-)included in the internal packages.
@gernotstarke gernotstarke added enhancement New feature or request codeClimate coding stuff labels Nov 18, 2024
@gernotstarke gernotstarke self-assigned this Nov 18, 2024
@gernotstarke gernotstarke moved this from Backlog to In progress in PDFMinion Dec 6, 2024
@gernotstarke
Copy link
Member Author

will add issues for every required flag/flag+param combo

@gernotstarke
Copy link
Member Author

still not good enough: I want a layered configuration: cli parameters -> (overwrites) config file -> defaults.

Therefore we add more sophisticated frameworks (e.g. viper) to handle such configuration.

gernotstarke added a commit that referenced this issue Dec 23, 2024
@gernotstarke
Merge branch '#15-update-config-handling'
884d97a 
# Conflicts:
#	docs/_pages/index.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gernotstarke gernotstarke

Labels
codeClimate coding stuff enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@gernotstarke