Skip to content

Conversation

@yufeih
Copy link
Contributor

@yufeih yufeih commented Nov 24, 2023

Fixes #9457

@yufeih yufeih added the bug-fix Makes the pull request to appear in "Bug Fixes" section of the next release note label Nov 24, 2023
@codecov
Copy link

codecov bot commented Nov 24, 2023

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (d68b9d2) 71.34% compared to head (6ec8634) 71.34%.

Files Patch % Lines
src/Docfx.Common/Loggers/ConsoleLogListener.cs 80.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9464   +/-   ##
=======================================
  Coverage   71.34%   71.34%           
=======================================
  Files         561      561           
  Lines       23987    23991    +4     
  Branches     4173     4173           
=======================================
+ Hits        17113    17116    +3     
- Misses       5779     5780    +1     
  Partials     1095     1095           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yufeih yufeih merged commit fc8404c into main Nov 24, 2023
@yufeih yufeih deleted the consoletextbreak branch November 24, 2023 05:47
p-kostov pushed a commit to ErpNetDocs/docfx that referenced this pull request Jun 28, 2024
fix: log message text break at 80 width

Co-authored-by: Yufei Huang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug-fix Makes the pull request to appear in "Bug Fixes" section of the next release note

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Warning message wraps at 80 columns, making MSBuild parse it wrong

2 participants