Skip to content

Conversation

@shvlv
Copy link
Contributor

@shvlv shvlv commented Feb 19, 2025

PHPCS config might contain <arg value="p"/>. In this case, we got invalid JSON like

$ vendor/bin/phpcs --report=json ./src/functions/boot.php 
. 1 / 1 (100%)


{"totals":{"errors":0,"warnings":0,"fixable":0},"files":{"\/src\/functions\/boot.php":{"errors":0,"warnings":0,"messages":[]}}}
Time: 72ms; Memory: 14MB

With the the -q option, we ensure progress information will be suppressed (see squizlabs/PHP_CodeSniffer#969).

@shvlv shvlv requested a review from borkweb February 19, 2025 14:50
@borkweb borkweb merged commit 70041d0 into main Feb 20, 2025
@borkweb borkweb deleted the fix/phpcs-progress branch February 20, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants