Could we unbloat the output a little?

[NickSdot]

Hey y'all 👋

I'd like to kick of a discussion. Currently fixer output looks something like this:

PHP CS Fixer 3.93.0 Exceptional Exception by Fabien Potencier, Dariusz Ruminski and contributors.
PHP runtime: 8.5.1
Loaded config default from ".php-cs-fixer.php".
Running analysis on 9 cores with 10 files per process.
Parallel runner is an experimental feature and may be unstable, use it at your own risk. Feedback highly appreciated!
Using cache file "/Users/Nick/Code/NickSdot/nicksdot/foo/storage/quality/.php-cs-fixer.cache".
 6/6 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

   1) /Users/Nick/Code/NickSdot/nicksdot/foo/app/Providers/AppServiceProvider.php

Fixed 1 of 6 files in 0.001 seconds, 22.00 MB memory used

sometimes when I don't obey even like this:

You are running PHP CS Fixer on PHP 8.5.1, but the minimum PHP version supported by your project in composer.json is PHP 8.4. 
Executing PHP CS Fixer on newer PHP versions may introduce syntax or features not yet available in PHP 8.4, which could cause 
issues under that version. It is recommended to run PHP CS Fixer on PHP 8.4, to fit your project specifics.                                                                                                                                                   
If you need help while solving warnings, ask at https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/discussions/, we will help you!

PHP CS Fixer 3.93.0 Exceptional Exception by Fabien Potencier, Dariusz Ruminski and contributors.
PHP runtime: 8.5.1
Loaded config default from ".php-cs-fixer.php".
Running analysis on 9 cores with 10 files per process.
Parallel runner is an experimental feature and may be unstable, use it at your own risk. Feedback highly appreciated!
Using cache file "/Users/Nick/Code/NickSdot/nicksdot/foo/storage/quality/.php-cs-fixer.cache".
 6/6 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

   1) /Users/Nick/Code/NickSdot/nicksdot/foo/app/Providers/AppServiceProvider.php

Fixed 1 of 6 files in 0.001 seconds, 22.00 MB memory used

That's 11-16 lines of cognitive bloat for each run. What I personally would need and love to see is:

6/6 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%
1) /Users/Nick/Code/NickSdot/nicksdot/foo/app/Providers/AppServiceProvider.php
Fixed 1 of 6 files in 0.001 seconds, 22.00 MB memory used

I saw the todos for 3.99 and 4.0, and I understand that you need a way to communicate some of these things. I also understand that credits are a thing.

I, however, wonder if you guys would be open to a minimal config to hide all these things. Or alternatively allow hiding everything individually. Communicating warnings could be addressed by adding them at the bottom of the help command, or if you want it more radical dump it to a file in the project root.

Personally, I don't need to see on each run that:

• I run 8.5 while composer is set to 8.4
• what fixer version is running
• who made it (no offense, big thanks!)
• what PHP version is running
• on what config it runs (especially not when I literally pass the flag)
• on how many cores it runs
• parallel runner is experimental (never had an issue since day one btw)
• a cache file is used

In before: the --quiet flag isn't helpful here, because it hides everything.

If there is any appetite to solve this in whatever way, I'd highly appreciate it and offer my help. I just would need to know which approach you maintainers could accept.

Thanks!

[keradus]

I run 8.5 while composer is set to 8.4

version mismatch is one of challenges we have and it can make your code not compatible with 8.4 anymore. highly discouraged.

what fixer version is running
who made it (no offense, big thanks!)
what PHP version is running
on what config it runs (especially not when I literally pass the flag)

I want to keep it. thanks.
Wording here can be shrinked, eg replace paths with relative ones.

on how many cores it runs
parallel runner is experimental (never had an issue since day one btw)

well, even last week i was fixing a bug related to parallel runner.
I like to get rid of this when we manage to enable the test suit under linux/mac/windows. right now, tests are not executed on each OS. do you like to give it a try ?

a cache file is used

agree. we can replace with notice if cache is not used.

---

Separately, i was also wondering on cleaning up the way to display those infos. Better separation of informative lines (Eg version) and warnings.

[NickSdot]

Wording here can be shrinked, eg replace paths with relative ones.

How would you feel about the following?

Fixer 3.93.0 on PHP 8.5.1 with .php-cs-fixer.php (9 cores × 10 files), no cache. <Warnings!> run `php-cs-fixer warn`.
By Fabien Potencier, Dariusz Ruminski & contributors. GitHub Sponsors welcome!
6/6 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100% (1/6 fixed · 0.001s · 22 MB)

   1) /Users/Nick/Code/NickSdot/nicksdot/foo/app/Providers/AppServiceProvider.php

• "Warnings" would be (as now), and only show when there are actual warnings. Detailed warning when running the new php-cs-fixer warn command.
• "no cache" is dynamic.
• "with .php-cs-fixer.php" could be dynamic and only show when not using standard config (how PHPStan does it). Thoughts?

So the very minimum, in most systems, and depending on what's the config, would be the following (given the parallel warning wouldn't move to docs).

Fixer 3.93.0 on PHP 8.5.1 (9 cores × 10 files). <Warnings!> run `php-cs-fixer warn`.
By Fabien Potencier, Dariusz Ruminski & contributors. GitHub Sponsors welcome!
6/6 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100% (1/6 fixed · 0.001s · 22 MB)

   1) /Users/Nick/Code/NickSdot/nicksdot/foo/app/Providers/AppServiceProvider.php

• No 'with .php-cs-fixer.php' because default.
• No ', no cache.' because active.

right now, tests are not executed on each OS.

You mean the macOS failures? Or expanding the Matrix? Is there an open issue or pr? Read the config comments--not entirely sure what you mean.

[keradus]

(OS topic moved to separated thread)

[keradus]

PHP CS Fixer 3.93.0 Exceptional Exception by Fabien Potencier, Dariusz Ruminski and contributors.

this line i want to keep as is. (explicitly towards your proposal - I want to keep full name of the tool and codename. codename is highlighting important elements from my life or celebrate important change in tool or core team. this is one of main happines i'm taking from this project)

You are running PHP CS Fixer on PHP 8.5.1, but the minimum PHP version supported by your project in composer.json is PHP 8.4.
PHP runtime: 8.5.1

Probably we can combine, but let's do so without dedicated new command. like
PHP runtime: 8.5.1 (⚠️ can introduce incompatibilities toward 8.4 from composer.json [read more])

Loaded config default from ".php-cs-fixer.php".

It can provide different results, eg:

• Loaded config PHP-LOWEST from ".php-cs-fixer.php-lowest.php".
• Loaded config default from "/Users/dariusz.ruminski/github/FriendsOfPHP/PHP-CS-Fixer/.php-cs-fixer.dist.php".

probably OK to give the config name only.

Cache

dynamic "no cache", good

parallel

externalized to different thread.

[NickSdot]

Understood! I'll have a stab at the implementation ASAP.

[keradus]

🧵 How to get rid of parallel warning

right now, tests are not executed on each OS.

You mean the macOS failures? Or expanding the Matrix? Is there an open issue or pr? Read the config comments--not entirely sure what you mean.

We need to remove this @requires and run the test on each OS. right now, it would cause CI to fail.
#9392

could you try to fix it up, @NickSdot ?

[keradus]

solved and removed some lines already:
https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/pull/9408/changes#diff-24bb82ed0664438ff09f690ca3d02d1c3bd7512c244854d24bdd7db9e9734102L350

[NickSdot]

Nice! Sorry I couldn't get on it, I am a bit occupied. Will surely follow up with the formatting when I have a free mind to get on it.

[keradus]

no worry, if you will have time to play with other part, fantastic. if not, i guess we get into it one day