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

Use configuration files instead of CLI arguments to configure the client and server apps #9

Open
rbost opened this issue Dec 22, 2018 · 0 comments

Comments

@rbost
Copy link
Member

rbost commented Dec 22, 2018

The number of CLI arguments, the poor documentation, and the difficult use of these arguments pledge for a change in the way the client and server applications are configured at startup.
A way better way to deal with configuration is to use configuration files, and to pass the configuration file as the only argument to the executables.

This would require to add code to parse this configuration file, but, as we already use rapidjson to parse JSON datasets, if we decide to use JSON as the format of the configuration file, we would not have to add an additional dependency to the code base. On the other hand, YAML or TOML might be better choices than JSON to write configuration files, but it would require to use a new library.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant