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

Colored output #9

Merged
merged 4 commits into from Sep 23, 2016
Merged

Colored output #9

merged 4 commits into from Sep 23, 2016

Conversation

@Apanatshka
Copy link
Contributor

Apanatshka commented Jul 26, 2016

I just started with implementing #7.
Right now it only does automatic detection and colouring, and I made the header row bold to make it more distinctive.

Still needs the flag, so don't merge yet.

@Apanatshka
Copy link
Contributor Author

Apanatshka commented Jul 26, 2016

Ok, the color flag was also simple to add.

@BurntSushi: ready for review.

@Apanatshka
Copy link
Contributor Author

Apanatshka commented Aug 20, 2016

@BurntSushi: Do you have time to review this? It's a small diff..

@Apanatshka
Copy link
Contributor Author

Apanatshka commented Sep 22, 2016

@BurntSushi: Since I have your attention, can you review this PR too? It's a tiny change :)

@BurntSushi BurntSushi merged commit 6ff3b5e into BurntSushi:master Sep 23, 2016
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@BurntSushi
Copy link
Owner

BurntSushi commented Sep 23, 2016

Let's do it. Thanks!

@Apanatshka Apanatshka deleted the Apanatshka:colored-output branch Sep 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.