-
Notifications
You must be signed in to change notification settings - Fork 10
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
Customize the directory name ".cyphr" #30
Comments
Thanks for the request - I will have a look and see |
Looks like this is unlikely to make it in before the next CRAN release (as CRAN have requested an update) but could you please elaborate on your use case here a bit. In particular, can you have any file called .cyphr? I am reluctant to use options here because then different cyphr repositories on a machine that use different conventions would require tweaking of options or changing the script in order to work. So I'd propose that we either store the cyphr files in If that does not work then my second proposal is a fixed set of names to look in ( If possible I would like to avoid an argument to the functions as the implementation is already complicated enough. Let me know what you think |
Hi, Yes, the ".cyphr" file is possible (I can make my R script to create it in the local files when the package is used) or a fixed fixed set of options works for me (I can suggest the "_cyphr" form, as it's often used a an dot notation alternative). I cannot use any name starting by a dot in our context because the files are shared using a nextcloud instance and we cannot share the dotfiles (it's only configurable at the account level and it will cause a big mess if we do that. But it's possible to use them on each user's sync copy. Yes, I perfectly understand the constraint to not change the function signature :) Many thanks |
We would like to use this package but it is not possible to use an hidden directory to store the credentials in our settings. Is it possible to make this configurable (with ".cyphr" as default value) via options() for example ?
The text was updated successfully, but these errors were encountered: