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

Detect and handle detached HEAD state #4

Merged
merged 7 commits into from Jul 19, 2018

Conversation

apm963
Copy link
Contributor

@apm963 apm963 commented Jul 16, 2018

Fixes #3

When a detached HEAD state is detected it echos ? Detached HEAD from git-status.sh:

 [git-extra-status] - HEAD
    ? Deteached HEAD x Uncommitted Changes

image

Please feel free to use a different symbol and color instead of a cyan question mark.

sandeep1995 and others added 7 commits July 11, 2018 10:23
First version 1.0.0 released.

* tag '1.0.0':
When running in my current environment (git bash for Windows) the script 
was failing with the following message:
```sh
$ ges
/c/Users/user-name/git/github/git-extra-status/bin/ges: line 1: 
../src/git-extra-status.sh: No such file or directory
```
This was because the relative path in the running script was being 
inherited from the shell's pwd, not of the script itself.
Without this, running `ges` with a dir name(s) as args would not be able 
to be read by the .sh scripts and would cause the command to silently 
fail
Use current script execution path to get relative path
@sandeep1995
Copy link
Owner

@apm963 Can you also send me a screenshot after running ges * in a directory where multiple git repos are present?

@sandeep1995 sandeep1995 added the enhancement New feature or request label Jul 17, 2018
@apm963
Copy link
Contributor Author

apm963 commented Jul 18, 2018

Here you go:

image

I redacted a few things with white boxes.

@sandeep1995 sandeep1995 merged commit 6e86aa2 into sandeep1995:master Jul 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants