Time and Memory info not printed

[llaville]

Hello Greg,

Thanks for your major version 2.0 (stable). It's a nice Christmas gift :)

BTW, I've noticed at least two minor errors that could be perharps fixed.

1 - When you check a data source, and it finished wiht at least a warning or an error, you got the Time / Memory line summary, but not where all finished well.

Platform : PHP 5.6.3
PHP_CodeSniffer version 2.0.0 (stable) by Squiz (http://www.squiz.net)

Example with error :

$ php phpcs.phar -p --standard=PSR2 /path/to/sources
............................................................ 60 / 61 (98%)
E

FILE: xxxxxxxxxxxxxxxxxxxxx
--------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------
 205 | ERROR | [x] The closing brace for the class must go on the next line
     |       |     after the body
--------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------

Time: 10.23 secs; Memory: 11.75Mb

Example without error :

$ php phpcs.phar -p --standard=PSR2 /path/to/sources
............................................................ 60 / 61 (98%)
.

Will be better to have

$ php phpcs.phar -p --standard=PSR2 /path/to/sources
............................................................ 60 / 61 (98%)
.
Time: 10.23 secs; Memory: 11.75Mb

I'll open a new report for 2nd minor issue

Thanks in advance
Laurent

[gsherwood]

I actually do this on purpose so that PHPCS is a quiet as possible when nothing is found. But it's actually a good point about the progress flag because PHPCS is always going to output something in this case, so it may as well just print the time and memory there as well.

Thanks for suggestion.

[gsherwood]

PHPCS now prints time and memory output even if no errors and shown, as long as progress information is being shown.

[llaville]

Thanks a lot for this fix and release 2.1.0