Kyle
Kyle
In our case `$this->fileName` is already a realpath and we may check if it's better to standardize output with `/` separator for relative path or use the `DIRECTORY_SEPARATOR` to respect...
Thanks @ttomdewit! Would you be OK to create a pull-request for that and ideally add a unit test that cover this new feature? So you'll get your name on this...
Thanks! In this PR we can focus on JSON renderer but I think some other renderers (like the XML one) may also benefit of this adding.
We have 3 different bugs here: Original bug of this stream: ```php /** * @SuppressWarnings(PHPMD.ShortMethodName) */ function ab() { return 1; } ``` Fixed. ```php class Model { /** *...
I confirm, this code should not trigger the violation. Thanks for the unit test, it will help to fix. 💪
I'm not sure we could (and should) fix this. I highly recommend the following syntax: ```php return array_map(function ($word) { return $this->foo($word); }, array('baz')); ``` I know, it's more verbose...
But it does not with the example of dynamic ones: https://psalm.dev/r/b86e124d51 (Even when this is actually static like in this example). And we won't implementing it just because an other...
Good point, so the options are: rewrite the rule using some `collect*` (manually count the `use` at the top of the file). Or edit PDepend to have some ignore filtering...
We will probably not have it in 2.8.0. It's not a top-priority for the core team, but pull-requests are welcome. And landing in 2.9.0 remains probable.
You can propose a PR. The probable entry-point for this feature is [src/main/php/PHPMD/Node/Annotation.php](https://github.com/phpmd/phpmd/tree/master/src/main/php/PHPMD/Node/Annotation.php)