[√] I have checked that another pull request for this purpose does not exist.
[√] I have considered, and confirmed that this submission will be valuable to others.
[ ] I have created or updated the relevant documentation for this change on the PHPCI Wiki.
[ ] Do the PHPCI tests pass?
Detailed description of change:
php_parallel_lint allows you to pass in a comma separated list of extensions to check but the PhpParallelLint plugin does not allow you to pass this to the command. It looks like the intention was to use a regexp option to do this, but it is not implemented and the regexp option is not used.
The changes in this pull request remove the regexp option (as it is unused) and allow you to set an extensions option in phpci.yml for the PhpParallelLint plugin.
Example
php_parallel_lint:
extensions: php, html
If the option is set to a comma delimited list of strings, it will pass that string through to the underlying php_parallel_lint command as a list of file extension to use. If the option is not set, or is not comma delimited strings, it will default to use php to preserve existing behaviour.
This is required for anyone that has php code in files with extensions other than php. This commonly occurs in templates where the file contains php but the extension is often .html, .phtml or .tpl
Contribution Type: new feature Link to Intent to Implement: Link to Bug:
This pull request affects the following areas:
In raising this pull request, I confirm the following (please check boxes):
Detailed description of change:
php_parallel_lint allows you to pass in a comma separated list of extensions to check but the PhpParallelLint plugin does not allow you to pass this to the command. It looks like the intention was to use a regexp option to do this, but it is not implemented and the regexp option is not used.
The changes in this pull request remove the regexp option (as it is unused) and allow you to set an extensions option in phpci.yml for the PhpParallelLint plugin.
Example
If the option is set to a comma delimited list of strings, it will pass that string through to the underlying php_parallel_lint command as a list of file extension to use. If the option is not set, or is not comma delimited strings, it will default to use
php
to preserve existing behaviour.This is required for anyone that has php code in files with extensions other than php. This commonly occurs in templates where the file contains php but the extension is often .html, .phtml or .tpl