-
Notifications
You must be signed in to change notification settings - Fork 261
Basic git improvements #1082
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
Basic git improvements #1082
Conversation
Thanks for updating it! Glad someone found it useful :) |
Looks like more than an update. Quite a bit has changed including less coloring. Is that right? |
@fdncred, yes, just made it look more like original output of |
I agree that there are some improvements here but I'm not really a fan of changing the coloring. That's the UI part for shells so it's kind of important. Some of these changes seem to be just preferences. It would be nice to include the original authors preferences but allow others to tweak to their own preferences. Otherwise, we lose most of the work the original author put into this. |
well, what can @Euphrasiologist say about this? Getting the original coloring back is not a problem |
The colours were what I personally wanted when I made it, and I agree it's nice if they are retained. But I'm happy either way 😊 |
okay, returned the branch color to original, but, as previous version doesn't consider staged and unstaged status, changes for status coloring seem reasonable. please take it into account, @fdncred |
I guess maybe I'm missing something because I don't see the original colors in the PR. yellow_bold and blue_bold aren't anywhere in it anymore. |
I am not sure it's a good design, but I tried to take the colors back |
Thanks |
Thanks to @Euphrasiologist for the prompt!
I found some bugs, and here are a few improvements.
git branch --show-current
instead ofgit rev-parse --abbrev-ref HEAD
to get current branch