Incorporate suggestions from #34

[mwilliammyers]

Incorporate suggestions from #34

• Simplify setting git_status
• Simplify __parse_git_branch by using echo instead of string

[rafaelrinaldi]

LGTM

[edouard-lopez]

@mwilliammyers Thanks for the follow up!

Would you be interested to add some tests related to your PR?

For instance on __parse_git_branch, you can write 2 test:

• works on default branch

  1. setup a git repo (before each test),
  2. check current branch is master

• works after checkout

  1. setup a git repo (before each test),
  2. create a new branch whatever
  3. checktout to whatever
  4. check branch name is whatever

[rafaelrinaldi]

@edouard-lopez This looks good to me as is but I also like your suggestions. Let's tackle this in a separate PR so this doesn't get blocked, ok?

Thanks again @mwilliammyers for sending this PR. This is a great addition.