-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
Don't emit ANSI escape codes with auto=false or simplified logging formats #755
Don't emit ANSI escape codes with auto=false or simplified logging formats #755
Conversation
Codecov Report
@@ Coverage Diff @@
## master #755 +/- ##
==========================================
+ Coverage 73.21% 73.30% +0.09%
==========================================
Files 48 49 +1
Lines 10835 10831 -4
==========================================
+ Hits 7933 7940 +7
+ Misses 2902 2891 -11
Continue to review full report at Codecov.
|
Would you like me to add a changelog entry for this? |
@josb if you don't mind doing so, that'd be great. It looks like dangerjs is struggling a little right now, but the check should pass once you add in a changelog entry. Feel free to add this into If you could do the same for your other PR (#754) that'd also be much appreciated. |
Added CHANGELOG.md entry. |
Thanks, @relaxolotl ! |
Sounds good! |
Fixes [#753].