phpro / grumphp

A PHP code-quality tool
MIT License
4.11k stars 430 forks source link

phpcs --file-list parameter causes incorrect results #987

Closed MelchiorKokernoot closed 1 year ago

MelchiorKokernoot commented 2 years ago
Q A
Version 1.8
Bug? no
New feature? no
Question? yes
Documentation? no

When running the Phpcs task with the --file-list parameter GrumPHP finds no errors, when commenting this line at:

phpro/grumphp/src/Task/Phpcs.php:87

$arguments->add('--file-list='.$tmpFile);

The output shows the expected errors:

My configuration

grumphp:
  parallel:
    enabled: true
    max_workers: 32
  stop_on_failure: true
  hide_circumvention_tip: true
  process_timeout: 240
  tasks:
    phpcs:
      standard: "phpcs.xml"
      triggered_by: [ "php"]

Steps to reproduce: vendor\bin\grumphp.bat run -> image

comment:

phpro/grumphp/src/Task/Phpcs.php:87

$arguments->add('--file-list='.$tmpFile);

vendor\bin\grumphp.bat run -> image

Result:

# Please add the result of the run or git commit actions here.

vendor\bin\grumphp.bat run -> image

comment:

phpro/grumphp/src/Task/Phpcs.php:87

$arguments->add('--file-list='.$tmpFile);

vendor\bin\grumphp.bat run -> image

veewee commented 2 years ago

Not sure what is going wrong, it could be a couple of things. Can you validate these?

veewee commented 1 year ago

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