2
78%
master: 79%

Ran 11 Dec 2023 06:49PM UTC

Files 257

Run time 9s

Badge

Embed ▾

Committed 11 Dec 2023 06:46PM UTC coverage: 63.088% (-0.02%) from 63.104%

Job # 7171974696.2

Build Type

push

github

Committed by

jrfnl

Commit Message

Report Full: fix two line wrapping bugs (#125)

* Fix two line wrapping bugs in default report formatter

The first is that the ANSI escape codes applied to bold the message when
`-s` is used were not being taken into account when wrapping the lines
for width, causing some lines to be wrapped unnecessarily.

The second is that when lines were wrapped in the middle of a long
message, the `|` characters making up the table were being bolded along
with the message.

* Report Full: iterate on line wrapping bug fix

This commit takes the fix one step further by adding the padding only after the message has been word-wrapped.
Includes correct handling of padding for multi-line error messages.

It then takes the last line of the resulting message and determines in isolation whether the source code suffix can fit on that line or needs to be placed on a new line.


Co-authored-by: Brad Jorsch <anomie@users.sourceforge.net>
Co-authored-by: jrfnl <jrfnl@users.noreply.github.com>

Run Details

14830 of 23507 relevant lines covered (63.09%)

1.82 hits per line

PHPCSStandards / PHP_CodeSniffer / 7171974696 / 2
78%
master: 79%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job php-8.3-custom-ini-false - 7171974696.2

PHPCSStandards / PHP_CodeSniffer / 7171974696 / 2 78% master: 79%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job php-8.3-custom-ini-false - 7171974696.2

PHPCSStandards / PHP_CodeSniffer / 7171974696 / 2
78%
master: 79%

README BADGES
x