Print abort and warn messages in red/magenta #924

Merged
merged 3 commits into from Jul 7, 2013

Projects

None yet

2 participants

@asmeurer
Contributor
asmeurer commented Jul 7, 2013

This makes them much easier to spot, and to notice when your fab failed.

This fixes (the easy part of) #923.

@asmeurer asmeurer Print abort and warn messages in red/magenta
This makes them much easier to spot, and to notice when your fab failed.

This fixes (the easy part of) #923.
cab4e7a
@asmeurer asmeurer referenced this pull request Jul 7, 2013
Closed

Color stderr #923

@asmeurer
Contributor
asmeurer commented Jul 7, 2013

I didn't add any tests for this. Should I?

@asmeurer
Contributor
asmeurer commented Jul 7, 2013

I just realized I didn't color error. Not sure how that should be done.

@asmeurer
Contributor
asmeurer commented Jul 7, 2013

Oh, never mind, it does color error, which apparently just calls abort (I didn't notice because I was on the wrong branch).

@bitprophet bitprophet added a commit that referenced this pull request Jul 7, 2013
@bitprophet bitprophet Tests re #924 f8b396e
@bitprophet bitprophet merged commit f0249ab into fabric:master Jul 7, 2013

1 check passed

default The Travis CI build passed
Details
@asmeurer
Contributor
asmeurer commented Jul 7, 2013

Thanks for the help!

@rlamy rlamy pushed a commit to rlamy/sympy that referenced this pull request Nov 3, 2013
@asmeurer asmeurer Enable colorized output from fabric
This requires my pull request fabric/fabric#924.
a8d5956
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment