Skip to content
This repository has been archived by the owner on Oct 20, 2023. It is now read-only.

Make GrGit exceptions return false result (issue #2) #3

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

scrain
Copy link

@scrain scrain commented Jun 24, 2016

This should resolve issue #2

@scrain
Copy link
Author

scrain commented Jun 24, 2016

Apologies, but not sure what that error is or what I might have done to cause it. Happy to amend if I could get a hint.

@DanielThomas
Copy link
Contributor

@scrain not you - we've been seeing this intermittently on some of our repositories on Travis. Thanks for the PR, will review it properly as soon as we're able.

@DanielThomas
Copy link
Contributor

In fact, I bet it's the clone depth - if the last commit to be tagged is outside of Travis' default clone depth, this will fail. Seeing the same failure on master, I'll fix that so you can rebase and get this PR built.

@DanielThomas
Copy link
Contributor

That should do it. Pull master from us and rebase onto your PR branch.

@scrain
Copy link
Author

scrain commented Jun 30, 2016

Just a bump on this for awareness. Think everything should be good now. Thanks!

@oesolutions
Copy link

Any update on getting this fixed? This PR is ancient and yet the bug still exists and caused us to miss the fact that some pushes were failing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants