-
Notifications
You must be signed in to change notification settings - Fork 51
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
git: You are not currently on a branch #84
Comments
That's weird! Definitely not intended. Interesting: I just tested it on my local environment with gpm Behind the scenes gpm uses This isn't necessarily a problem: as long as the following line is present it means the commit you requested is in the local history and has been set.
Although... I do agree its ugly. 😅 Would it be worthwhile to add code to solve this warning? I'll think of a way around it, off the top of my head I can think about storing the default branch and checking it out before the update, or simply suggest deleting the directory and fetching it again. I could also simply delete it automatically, but I'm not big on deleting things without explicit user conscent. What do you think? |
Thanks for coming back. I think it is a good idea to checkout the default branch, do the |
Let's not go the delete route. What @xh3b4sd suggested in his last comment looks good. |
I found
gpm
and wanted to give it a try. It looks like it does not support pinning dependencies based on commits. I have this in myGodeps
file.This is what I get.
Is this intended? Can we make it work?
The text was updated successfully, but these errors were encountered: