gs fixes

[yhpark]

• Branch names containing dots (.) are now correctly parsed.
  (although not sure why '.' was in the regex in the first place.)
• Color/bold style is now correctly applied.
• Fixed a bug where no-change status is not detected.

before:

after:

[ghthor]

Thanks, this looks fine.

[ghthor]

I don't think there is a functional change here, but I wonder why this was left as a string comparison.

[yhpark]

@ghthor I just found there's an untended change introduced from this fix: if the branch is remote-tracking, git status --porcelain -b prints ## {local_branch}...{remote_branch} so the branch name is now parsed like branch...origin/branch. For me this is fine but if you want it the way before, changing the regex to something like ^## (?:Initial commit on )?(.*?)(?:\.\.\.| |$) would work.

ref: git doc says branch name cannot contain two or more consecutive dots.

[ghthor]

I think the change is fine. Just to clarify with some output. The change to the @branch variable is purely for the output of gs and I think showing the remote tracking branch as part of the output is an improvement here.

From this master

$ gs
# On branch: master |  [*] => $e*
#
➤ Untracked files
#
#      untracked: [1] lib/fun.sh 
#

To This master...origin/master

$ gs
# On branch: master...origin/master  |  [*] => $e*
#
➤ Untracked files
#
#      untracked: [1] lib/fun.sh 
#