Broadcoder

Results 9 comments of Broadcoder

Thank you for your fast answer, from what I can tell it happens for every Sniff class, so right now I set up ecs.php like this: ```php

Hello again and sorry to bother you again, but I am kind of stuck. I started creating a test project, only intalling ecs. I created a single file with 2...

Here is the project, maybe you can reproduce the issue on your side with it: https://github.com/broadcoder/ecs_sniffer_test Check out the project, run composer install, run composer run test:lint There is a...

@TomasVotruba I updated it, note how the pipeline says everything is fine, while it should complain about ExampleClass.php I hope it helps, thank you in advance 🙂

@TomasVotruba Yeah, rector is awesome, I would like to integrate, but it needs a code fixer tool as well and if someone uses rector but don't run fixes, the pipeline...

Yes, we use it with --dry-run, however for me it only reports diffs - I would like to have something like: Use null safe operator (?->) instead of trinary in...

But sniffs are not "fixing" from php 7 to 8 or am I wrong? Wouldn't it be good, to see a message, what could be updated without having a diff,...

> Not sure, but for reporting in ECS, you need sniffs that only add message, but do no change. Are you referring to the idea of reporting for example a...