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

Add ability to get tag of current head #54

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

btruhand
Copy link

I wanted to add ability to get the tag at HEAD (not the closest tag). Nothing big

@AaronDMarasco-VSI
Copy link

I haven't looked at the source lately, but if the "check if git (c726016)" isn't already in there, it should probably be at the beginning of everything and just shortcut when not.

@btruhand
Copy link
Author

btruhand commented Jun 4, 2017

@AaronDMarasco-VSI
Thanks for that input. Added it in to do some short-circuting. Sorry for the late reply didn't take a look at it past few days haha

prompt.sh Outdated
if branch=$(git rev-parse --abbrev-ref HEAD 2> /dev/null); then
if [[ "$branch" == "HEAD" ]]; then
branch='detached*'
echo $is_in_git

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this leftover debug code?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah yeah it is. Wow I've been running it myself but didn't notice. Guess I forgot to commit the removal. Will do tomorrow

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh forgot to message, I removed the debugging already

@dafeder
Copy link

dafeder commented Aug 14, 2019

There is a PR already adding tags here: #36 - suggest that the git checking stuff be separated into its own PR to keep things focused :)

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