PHPCS not running on grumphp

[prafullazee]

Q	A
Version	1.15.0
Bug?	maybe
New feature?	no
Question?	yes
Documentation?	no

phpcs: standard: ["PSR12", "PSR2", "PSR1", "PEAR"] ignore_patterns: [./tests/, ./resources/, ./database/, ./bootstrap/, ./storage/*] whitelist_patterns:

• /^app\/(.*)/ tab_width: 4 exclude: ['Squiz.WhiteSpace.OperatorSpacing', 'Squiz.Commenting.FileComment', 'Generic.Commenting.DocComment', 'PEAR.Commenting.FileComment', 'Squiz.Files.FileExtension', 'PEAR.Commenting.ClassComment', 'Squiz.Commenting.ClassComment', 'MySource.Channels.IncludeSystem', 'Squiz.Commenting.ClosingDeclarationComment', 'Squiz.WhiteSpace.FunctionSpacing', 'Squiz.WhiteSpace.MemberVarSpacing', 'Squiz.WhiteSpace.FunctionClosingBraceSpace', 'MySource.PHP.ReturnFunctionValue', 'MySource.Commenting.FunctionComment', 'PEAR.Commenting.FunctionComment', 'PEAR.NamingConventions.ValidVariableName'] severity: ~ error_severity: ~ warning_severity: ~ report: full report_width: ~ encoding: ~ sniffs: [] triggered_by: [php] show_sniffs_error_path: true

Steps to reproduce:

• run vendor/bin/grumphp git:pre-commit

Run GrumPHP:

./vendor/bin/grumphp run

**Result:**

Running task 1/15: file_size... ✔ Running task 2/15: composer... Running task 3/15: git_branch_name... ✔ Running task 4/15: phpversion... ✔ Running task 5/15: phpcs... Running task 6/15: phpcpd... ✔ Running task 7/15: phpstan... ✔ Running task 8/15: phplint... ✔ Running task 9/15: phpcsfixer2... ✔ Running task 10/15: phpspec... ✔ Running task 11/15: phpparser... ✔ Running task 12/15: phpmd... Running task 13/15: phpmnd... Running task 14/15: jsonlint... Running task 15/15: stylelint...

[rajuguptahipi]

When running the grumphp pre commit hook (./vendor/bin/grumphp:git:pre-commit), it fails to check for phpcs problems in php Laravel project. View the outcome above, The error isn't being validated by phpcs.

OS : Windows Editor: VScode

[veewee]

@rajuguptahipi phpcs only runs on the changed files. If no files are left after applying the ignore patterns, it will not run at all during that pre-commit. Can you make sure that is not the case here?

Since the list of files can be rather long, we pass the files through a temporary file. Maybe that is where it goes wrong? See

• https://github.com/phpro/grumphp/blob/master/src/Task/Phpcs.php#L85
• https://github.com/phpro/grumphp/blob/master/src/Process/TmpFileUsingProcessRunner.php

[prafullazee]

@veewee in my case, files are newly created and added to stage, still having the issue and if you have noticed phpcs, phpmd, phpmnd etc are not running and all others are running.

[veewee]

Well hard to tell from my side what is going wrong exactly on your setup with the information given. There are some things you can do to debug:

• Disable parallel execution and see if the issue is there as well.
• Run with -vvv flag (which displays the full executed commands), and validate if it is a tooling issue by running that exact command seperately.

[veewee]

Closing this one because of inactivity If the issue still persists - feel free to reopen and provide some additional context.