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

Replace config.json with config.yaml #38

Open
Deekee62 opened this issue Dec 20, 2021 · 2 comments
Open

Replace config.json with config.yaml #38

Deekee62 opened this issue Dec 20, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@Deekee62
Copy link

This is a suggestion only :-)

Unfortunately the JSON file format doesn't allow any comments and I would like to suggest the use of YAML

WHY?
I find it laboriously as a noob to keep separate notes and would appreciate it very much to have the possibility to comment things I change in config.json, as my old brain is not so fit anymore :-)

Especially the entries for "avoid" and "exclude" which are pubkeys.

@itsneski itsneski added the enhancement New feature or request label Dec 21, 2021
@itsneski
Copy link
Owner

I like the idea of having comments in the config file. Collecting feedback from the alpha-testing group.

@cameri
Copy link
Contributor

cameri commented Dec 31, 2021

There's a library that allows reading json with comments: https://www.npmjs.com/package/comment-json

There's also JSON5.

Perhaps it's worth considering these options unless YAML is preferred.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants