Dezső BICZÓ

Results 141 comments of Dezső BICZÓ

Okay, so this is not yet implemented :) (I have just had a need for something similar.)

Hi @elvetemedve! I completely agree about your reasoning regarding prioritizing unit testing over e2e testing. It is also completely understandable if you do not want to maintain this project anymore...

Let me copy the code snippet as an example from my issue that was flagged as duplicate of this one. https://phpstan.org/r/465f4f45-2dc7-4406-b82c-a21926d1bdc5

Please reopen @divya-intelli @phdhiren

Probably it is a good idea tagging this with "needs help".

Null paths can be and probably should be handled at the source: `\DrupalComposer\PreservePaths\PluginWrapper::getInstallPathsFromPackages()` ``` diff --git a/src/PluginWrapper.php b/src/PluginWrapper.php --- a/src/PluginWrapper.php +++ b/src/PluginWrapper.php (date 1710258587092) @@ -136,6 +136,8 @@ $paths[] =...

We would need something like this exactly in Drupal also. Did some digging and maybe if the excluded list of directories could be also specified in the `CodeSniffer.conf` file (https://github.com/squizlabs/PHP_CodeSniffer/blob/3.4.2/CodeSniffer.conf.dist)...

(Showcased on a screen share, please let me know if there is still an issue with reproducing it.)

Tested the fix on those false positives that we got reported and the patch did silence them. RTBC (The pipeline looks broken at this moment, I made an attempt to...