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.

Sign up for GitHub

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

Jump to bottom

Doesn't work in detached HEAD state #18

Closed
mrPjer opened this issue Feb 8, 2016 · 0 comments · Fixed by #17
Closed

Doesn't work in detached HEAD state #18

mrPjer opened this issue Feb 8, 2016 · 0 comments · Fixed by #17

Comments

@mrPjer
Copy link
Contributor

mrPjer commented Feb 8, 2016

It seems that this utility does not work when the repository is in detached HEAD state.

Running:

git checkout @~0
git hours

Results in:

Error: Reference 'refs/remotes/(HEAD/HEAD' not found
    at Error (native)

This is the reason why Travis CI builds fail with Uncaught SyntaxError: Unexpected end of input, which you reported in travis-ci/travis-ci#2493 and this can be confirmed by printing out stderror in any of the tests (the above error is displayed).
mrPjer added a commit to mrPjer/git-hours that referenced this issue Feb 8, 2016
@mrPjer
Use nodegit to fetch branch names
d416e28 
This should be more robust than using scripts and it also works when the
repository is in detached HEAD state.

The repository references are explicitly filtered to return only local
branches.

This fixes kimmobrunfeldt#18.
mrPjer added a commit to mrPjer/git-hours that referenced this issue Feb 8, 2016
@mrPjer
Use nodegit to fetch branch names
30fb8ed 
This should be more robust than using scripts and it also works when the
repository is in detached HEAD state.

The repository references are explicitly filtered to return only local
branches.

This fixes kimmobrunfeldt#18.
@mrPjer mrPjer mentioned this issue Feb 8, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update nodegit dependency and fix Travis build mrPjer/git-hours
1 participant
@mrPjer