-
Notifications
You must be signed in to change notification settings - Fork 82
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
Package-assisted path handling, via @EdwinTh #52
Comments
A way to describe the intent of All the processing of |
I am not only compliant, I am also very easily tempted... why not force users to follow the same convention within their home dir? have a helper function in the package that interactively creates a symlink A way to describe the intent of INTERNAL_PACKAGE::save_as_rds(): it's just like here::here() |
Yes, exactly.
I think it's an advantage that you don't have to edit the source of the package each time a team member comes on board or decides to rename a folder. As it stands, your package source actually needs to change if I am user2 and I decide to keep the data in "cuddly_brocolli/fluffy_otter/" instead of "cuddly_brocolli/lively_hamster". Ditto for when user3 and user4 join the team. |
Thanks @EdwinTh for blogging your approach!
Use to enrich discussion about ways to make paths more robust across users and computers.
https://edwinth.github.io/multiperson-project/
@hadley also got @EdwinTh's blessing to do a mini code review & makeover, which is here:
tidyverse/design#56
Repeating a comment I made there, in case I can tempt @EdwinTh into a discussion of it here:
Yeah, why not force users to follow the same convention within their home dir? Or have a helper function in the package that interactively creates a symlink from each user's idiosyncratic data storage choice to
~/project_name/data
. A user would do this once, when they start working on the project.Basically get the varying code / logic out of the package itself and just make sure each user's local situation can "explain itself" to the package.
The text was updated successfully, but these errors were encountered: