6
78%
master: 79%

Ran 24 Feb 2025 11:24AM UTC

Files 262

Run time 13s

Badge

Embed ▾

Committed 24 Feb 2025 11:16AM UTC coverage: 2.128%. Remained the same

Job # 13496759415.6

Build Type

push

github

Committed by

jrfnl

Commit Message

Runner: don't prefix Ruleset error notices

As things are, the `Runner` class catches any exception thrown by the Ruleset, enhances the message with an `ERROR` prefix, adds new lines and short help usage info and then re-throws this as a `DeepExitException`.

In an upcoming PR, a new `MsgCollector` class will be introduced, which will allow for both errors, as well as warnings, notices and deprecations.

This means that the `ERROR` prefix won't always be appropriate anymore.

This commit moves the `ERROR` prefix addition from the `Runner` class to the individual messages for the Exceptions being thrown in the `Ruleset` class.

Additionally, it changes the "trailing new lines" handling as the `MsgCollector` may also add new lines at the end of a message, as messages will not necessarily always be displayed via an exception (think: non-blocking deprecation notices).
So to prevent a duplicate set of new lines, any new lines which are included in the exception are trimmed off before adding the new lines desired for the Exception display.

_This is a preliminary step before introducing the `MsgCollector` to the `Ruleset` class._

Run Details

649 of 30491 relevant lines covered (2.13%)

0.22 hits per line

PHPCSStandards / PHP_CodeSniffer / 13496759415 / 6
78%
master: 79%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job os-windows-latest-php-7.2-custom-ini-false - 13496759415.6

PHPCSStandards / PHP_CodeSniffer / 13496759415 / 6 78% master: 79%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job os-windows-latest-php-7.2-custom-ini-false - 13496759415.6

PHPCSStandards / PHP_CodeSniffer / 13496759415 / 6
78%
master: 79%

README BADGES
x