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

Make the pre-push hook check that git-lfs exists first #339

Merged
merged 3 commits into from
May 28, 2015
Merged

Conversation

rubyist
Copy link
Contributor

@rubyist rubyist commented May 27, 2015

If git-lfs does not exist (e.g. it has been uninstalled) the pre-push hook should still exit cleanly so the push can continue. The script will output a warning stating that the repo has been set up for Git LFS but that it is not installed.

/cc #333

If git-lfs does not exist (e.g. it has been uninstalled) the pre-push
hook should still exit cleanly so the push can continue. The script will
output a warning stating that the repo has been set up for Git LFS but
that it is not installed.
@technoweenie
Copy link
Contributor

👍 Though unfortunately the command is now complex enough that we can't just copy and paste it around the tests.

rubyist added a commit that referenced this pull request May 28, 2015
Make the pre-push hook check that git-lfs exists first
@rubyist rubyist merged commit 5ebb953 into master May 28, 2015
@rubyist rubyist deleted the prepush branch May 28, 2015 16:04
@technoweenie technoweenie mentioned this pull request May 29, 2015
38 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants