PHPCSStandards / PHPCSExtra / 173

99%

develop: 100%

Pull #43

travis-ci-com

:sparkles: New `Universal.WhiteSpace.DisallowInlineTabs` sniff

New sniff to enforce using spaces for mid-line alignment.

While tab versus space based indentation is a question of preference, for mid-line alignment, spaces should always be preferred, as using tabs will result in inconsistent formatting depending on the dev-user's chosen tab width.

> _This sniff is especially useful for tab-indentation based standards which use the `Generic.Whitespace.DisallowSpaceIndent` sniff to enforce this._
>
> **DO** make sure to set the PHPCS native `tab-width` configuration for the best results.
> ```xml
>    <arg name="tab-width" value="4"/>
> ```
>
> The PHPCS native `Generic.Whitespace.DisallowTabIndent` sniff (used for space-based standards) oversteps its reach and silently does mid-line tab to space replacements as well.
>
> However, the sister-sniff `Generic.Whitespace.DisallowSpaceIndent` leaves mid-line tabs/spaces alone.
> This sniff fills that gap.

Implementation notes:
* Includes a `DummyTokenizer` to allow use of the PHPCS native intelligent tabs to spaces replacement using the default tab-width if none was set.

Includes fixers.
Includes unit tests.
Includes documentation.

36 of 36 new or added lines in 1 file covered. (100.0%)

633 of 637 relevant lines covered (99.37%)

3.96 hits per line