This PR can be summarized in the following changelog entry:
Use Parallel Linting
Relevant technical choices:
Composer
This installs two additional PHP packages in require-dev:
php-parallel-lint which allows for linting PHP files in parallel (faster), as well as automatically recursively walking directories.
php-console-highlighter which provides PHP code highlighting in the command line console, allowing the linter to display the results in a more meaningful manner.
It also adds a new lint script for use with Composer.
Travis
Switch out the script part in the Travis script which did the linting the "old-fashioned" way to use the new Parallel linting option.
Adjust the composer install command in the install section to always run.
Note: for PHP "nightly" we need to ignore-platform-reqs for the time being as one of the PHPCS related dependencies does not allow for installation on PHP 8 yet.
This has been fixed in the dependency, but there hasn't been a release yet containing the fix.
Includes removing an outdated comment in the Travis script.
Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
Composer
This installs two additional PHP packages in
require-dev
:php-parallel-lint
which allows for linting PHP files in parallel (faster), as well as automatically recursively walking directories.php-console-highlighter
which provides PHP code highlighting in the command line console, allowing the linter to display the results in a more meaningful manner.It also adds a new
lint
script for use with Composer.Travis
composer install
command in theinstall
section to always run.ignore-platform-reqs
for the time being as one of the PHPCS related dependencies does not allow for installation on PHP 8 yet. This has been fixed in the dependency, but there hasn't been a release yet containing the fix.Includes removing an outdated comment in the Travis script.
Ref:
Test instructions
This PR can be tested by following these steps:
Testing this PR
composer install
.composer lint
& see it in action.composer lint
& see the error being reported.Also check a couple of the Travis builds to verify that the linting is running and passing.