-
Notifications
You must be signed in to change notification settings - Fork 19
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
Finalize option interface #264
Comments
Actually seeing this, maybe telemetry should not be one file per option as in my PR but instead there could be a duckplyr configuration file? https://blog.r-hub.io/2020/03/12/user-preferences/#using-a-config-file |
This way, all options would be in one place. Maybe it could somehow be created by default, to show the default options. |
The setting of options could happen via functions, or manually. There could be several locations for the configuration file like for an Having it all in one file means it's easier for users to get an overview but probably also for us to document all options somewhere central? |
Yeah, the user interface vs. the actual storage of the options are two separate questions. I'll think about it and work with what's in the PRs. |
Descoping, the env var interface is good enough. |
Support options in addition to (or instead of) env vars.
The text was updated successfully, but these errors were encountered: