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

Support a simpler config file format #385

Open
x4e-jonas opened this issue Jul 19, 2023 · 2 comments
Open

Support a simpler config file format #385

x4e-jonas opened this issue Jul 19, 2023 · 2 comments
Labels
enhancement Good First Issue Good Issue for first-time contributors

Comments

@x4e-jonas
Copy link
Contributor

JSON is hard to read and write for humans. E.g. you can't just diff/merge lines of config and it does not support comments.

I'd suggest TOML since it's a very simple standardized format and Java implementations are already available. Other candidates are YAML, INI or HCL like formats.

@lucaro
Copy link
Member

lucaro commented Jul 19, 2023

While the JSON standard does not include comments, many parsers, including the one cineast uses, support C-style comments, so you can use comments if you want. TOML could potentially be an alternative if one wanted. I'm not a bit fan of INI and YAML, as they are somewhat limited and/or cumbersome. There's also HOCON which, being a superset of JSON would have the advantage that configurations would even stay backwards compatible.

@lucaro lucaro added enhancement Good First Issue Good Issue for first-time contributors labels Jul 19, 2023
@lezhdz98
Copy link

@x4e-jonas I would like to help you with this issue, the config file is extraction_config.json ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Good First Issue Good Issue for first-time contributors
Projects
None yet
Development

No branches or pull requests

3 participants