Renamed variables to reflect test results#23
Conversation
|
@subtlepseudonym who is responsible for the actual merge of the PR? |
|
There are some conflicts now, can you please resolve? Otherwise, LGTM. |
|
I do not know if it was a Git hiccup but the conflict did not capture the change in line 104-105 with the color handling for |
|
Thanks! |
|
I was looking at the release history and it seems my PR broke something, which resulted in release 0.0.4. I am not sure as too what broke and why, I was very careful with my first PR and I tested thoroughly before submitting, but I am a Go n00b and I might have missed something. I hope it has not caused too much inconvenience, I have updated my other PRs from master, where applicable and have done some more testing. Anyway thanks for accepting my PR, it means a lot. |
Renamed the variable to reflect the states of the tests.
This request originates from the dialogue with @subtlepseudonym and review of PR #18
I hope you find it useful.