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

Parse commit from packed refs if not available in refs dir. #163

Merged
merged 4 commits into from
Mar 30, 2017

Conversation

chasdevs
Copy link
Contributor

@chasdevs chasdevs commented Mar 29, 2017

In circle-ci I was getting failures due to their git clone process not providing a reference in .git/refs/heads/master. If that is not available, get the commit hash from .git/packed-refs.

@coveralls
Copy link

coveralls commented Mar 29, 2017

Coverage Status

Coverage increased (+0.5%) to 95.77% when pulling 06f646f on cdeveas:parse-commit-from-packed-refs into e476964 on nickmerwin:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.5%) to 95.77% when pulling c84d09e on cdeveas:parse-commit-from-packed-refs into e476964 on nickmerwin:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.5%) to 95.77% when pulling c84d09e on cdeveas:parse-commit-from-packed-refs into e476964 on nickmerwin:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.5%) to 95.77% when pulling c84d09e on cdeveas:parse-commit-from-packed-refs into e476964 on nickmerwin:master.

@coveralls
Copy link

coveralls commented Mar 29, 2017

Coverage Status

Coverage increased (+0.5%) to 95.77% when pulling 091dae2 on cdeveas:parse-commit-from-packed-refs into e476964 on nickmerwin:master.

@nickmerwin
Copy link
Owner

This looks great, thanks @cdeveas!

@nickmerwin nickmerwin merged commit ef7e811 into nickmerwin:master Mar 30, 2017
@chasdevs chasdevs deleted the parse-commit-from-packed-refs branch March 30, 2017 17:11
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.

3 participants