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

Is there appetite to extend Use Git button to using git lfs as well? #40

Open
DaveParr opened this issue Feb 14, 2018 · 8 comments
Open

Comments

@DaveParr
Copy link

The package currently calls to devtool::use_git() when the Use git option is TRUE. The package also contains the option of including a data folder. git lfs (large file storage) is a useful extension to git that indexes file by type/location, and commits them to git in a safe way that doesn't lock up the repo when it's hosted externally, or slow it down if a large data set changes a lot.

Would it be technically possible to have a use git lfs button? I don't believe devtools has a wrapper to it, so any solution will probably have to be implemented slightly differently. If it were possible, would it be useful/desired?

@stephlocke
Copy link
Member

That's a nifty idea! It's pretty easy to setup tracking for common data extensions like csv sqlite etc
We could use git2r to execute the commands

It'll have to be an opt-in but it's a great idea

@maelle
Copy link
Contributor

maelle commented Jul 9, 2018

for large files & GitHub see also https://github.com/cboettig/piggyback

@maelle
Copy link
Contributor

maelle commented Oct 23, 2018

@DaveParr could you explain more precisely what you'd want? Thanks!

@stephlocke
Copy link
Member

Not sure about Dave but I frequently want datasets or have outputs that bust the max file size for github - setting it up retrospectively is often cripplingly hard and I'd welcome it being "on by default"

@maelle
Copy link
Contributor

maelle commented Oct 23, 2018

In practice, does this mean gitignoring+lfs-ing a folder by default? which folder?

@stephlocke
Copy link
Member

possibly the data folder and outputs folders or having a set of default extensions we'll LFS as part of the fn args

@maelle
Copy link
Contributor

maelle commented Oct 23, 2018

Ok. When I get to this, I'll probably do some reading and more questioning. 😁

@DaveParr
Copy link
Author

That's a pretty accurate summary of what I had in mind :)

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

3 participants