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

Replace TOML for the config with ... something #27

Closed
bep opened this issue Aug 28, 2022 · 0 comments
Closed

Replace TOML for the config with ... something #27

bep opened this issue Aug 28, 2022 · 0 comments
Labels
thinking

Comments

@bep
Copy link
Member

bep commented Aug 28, 2022

When I started this project, part of the motivation was to see how it would look if I started a new CLI project from scratch.

I chose TOML as the config format because I like its simplicity; it's easy to read (especially when properly indented) and is robust against typing mistakes (YAML errors can be relatively costly when you don't discover them before the CI build is running).

But, looking back (and also after discovering aliases in YAML), especially at the duplication/merge logic added to avoid too much repetition, I want to take a minute to think about better alternatives out there before I set this in stone.

Maybe ...

/~https://github.com/cue-lang/cue
@bep bep added the thinking label Aug 28, 2022
@bep bep assigned bep and unassigned bep Aug 28, 2022
bep added a commit that referenced this issue Sep 3, 2022
@bep
readme: Remove the remark about switching config framework
f2e8453 
* I'm pretty happy with the current setup
* Going with something ala Cuelang would probably be a little on the advanced side for such a small project
* Also, I don't have the time right now do do the research work

Closes #27
bep added a commit that referenced this issue Sep 3, 2022
@bep
readme: Remove the remark about switching config framework
cd636f6 
* I'm pretty happy with the current setup
* Going with something ala Cuelang would probably be a little on the advanced side for such a small project
* Also, I don't have the time right now do do the research work

Closes #27
@bep bep closed this as completed in d770a7a Sep 3, 2022
bep added a commit that referenced this issue Sep 3, 2022
@bep
readme: Remove the remark about switching config framework
9cdcec3 
* I'm pretty happy with the current setup
* Going with something ala Cuelang would probably be a little on the advanced side for such a small project
* Also, I don't have the time right now do do the research work

Closes #27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
thinking
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@bep