-
Notifications
You must be signed in to change notification settings - Fork 41
Open
Labels
cat:refactorrefactor/clean up/simplifications/etc.refactor/clean up/simplifications/etc.
Description
Is your feature request related to a problem?
We currently use toml11 library to load TOML configuration files.
In case you didn't know, you should not use `JSON for configuration files](https://www.reddit.com/r/programming/comments/bd0jsl/json_as_configuration_files_please_dont/).
Description
Although we don't use TOML configuration files with great extend currently, we have some issues with toml11.
- Cloning the toml11 dependency takes a very long time because it always clones the git submodules of the testing enviroment and some more stuff. This takes significantly longer than everything else in the CMake setup, which also affects build times in CI.
- We cannot disable cloning the submodules, even though we are not building the tests of the toml11 project!
- It is not as actively maintained as tomlplusplus.
Alternatives
Use https://github.com/marzer/tomlplusplus
Affected Code
All code that works with TOML configuration files (currently only renderer setup).
Operating System
all
Additional Context
none
Metadata
Metadata
Assignees
Labels
cat:refactorrefactor/clean up/simplifications/etc.refactor/clean up/simplifications/etc.