• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

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

Build:
Build:
LAST BUILD BRANCH: catch-per-3.0
DEFAULT BRANCH: master
Ran 27 Nov 2024 08:25PM UTC
Files 262
Run time 8s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

27 Nov 2024 08:19PM UTC coverage: 0.178%. Remained the same
12057684915.1

push

github

jrfnl
Generic/OpeningFunctionBraceKernighanRitchie: improve error message

Follow up on 707

When there are multiple tabs between the end of the function signature and the open brace, the "found: $length" part of the error message would refer to the number of tabs found.

This is inconsistent with other sniffs and confusing.

This commit fixes this, while still maintaining the previous behaviour - as introduced via https://github.com/squizlabs/PHP_CodeSniffer/commit/c4b9807cd - of special casing the messaging for a single tab - independently of whether or not tab replacement is in effect.

Example for line 10 of test case file 2:
```
// Before:
Expected 1 space before opening brace; found 3

// After:
Expected 1 space before opening brace; found 11
```

Covered via pre-existing tests in both test case files + some additional new tests.

58 of 32631 relevant lines covered (0.18%)

0.0 hits per line

Source Files on job cbf-os-ubuntu-latest-ubuntu-latest-php-7.2-custom-ini-false - 12057684915.1
  • Tree
  • List 0
  • Changed 1
  • Source Changed 1
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 12057684915
  • b282c5f2 on github
  • Prev Job for on 4.0 (#12048293772.1)
  • Next Job for on 4.0 (#12091631222.3)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc