1
78%
master: 79%

Ran 09 Jul 2024 05:14AM UTC

Files 260

Run time 5s

Badge

Embed ▾

Committed 09 Jul 2024 05:09AM UTC coverage: 69.685% (-0.005%) from 69.69%

Job # 9851335454.1

Build Type

push

github

Committed by

jrfnl

Commit Message

Generic/DisallowTabIndent: do not auto-fix heredoc/nowdoc closer indent

Follow up on squizlabs/PHP_CodeSniffer 3640, which added the `T_END_HEREDOC` and `T_END_NOWDOC` tokens to the list of tokens to examine and was included in the 3.7.2 release.

When a flexible heredoc/nowdoc is used, the indentation type of the heredoc/nowdoc content and the indentation type of the heredoc/nowdoc closer must be consistent, so auto-fixing the indentation of the closer, without also changing the indentation of the contents (from tabs to spaces) would cause a parse error.
See: https://3v4l.org/7OF3M

I do believe indentation tabs should still be flagged, but the auto-fixer should be disabled.

To allow people to disregard tab indentation for heredoc/nowdoc closers completely, the error now also has a more specific error code - ` TabsUsedHeredocCloser` -, which allows for excluding it in a ruleset.

While the change in the error code _could_ be considered a breaking change, the fact that the potential for a parse error hasn't been reported as a bug so far, leads me to believe this is not a frequently encountered situation, so the impact of the higher specificity for the error code should be minimal.

Run Details

16921 of 24282 relevant lines covered (69.69%)

23.71 hits per line

PHPCSStandards / PHP_CodeSniffer / 9851335454 / 1
78%
master: 79%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job php-7.2-custom-ini-false - 9851335454.1

PHPCSStandards / PHP_CodeSniffer / 9851335454 / 1 78% master: 79%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job php-7.2-custom-ini-false - 9851335454.1

PHPCSStandards / PHP_CodeSniffer / 9851335454 / 1
78%
master: 79%

README BADGES
x