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

comments are part of the section name? #11

Open
Dieterbe opened this issue Jun 17, 2021 · 0 comments
Open

comments are part of the section name? #11

Dieterbe opened this issue Jun 17, 2021 · 0 comments

Comments

@Dieterbe
Copy link

looks like [foo] # comment here creates a section with fqn foo] # comment here
I see how this helps with serializing the config back to a file (losslessly) but intermingling extraneous comment data in the name seems problematic.

I notice a similar issue with empty lines / whitespace lines being saved as an option with key and val set to "". i see how this helps make the file reproduceable (although i'm not sure if it works well with repeated lines), but from a "parsing config" perspective, I think this muddies the config.

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

No branches or pull requests

1 participant