Kyle
Kyle
Even when running the CLI applications, having visible notices that come when you run a method/read a setting would mean that when you will execute a phpmd command, you could...
Actually we basically detect only methods called with `::` or `->`. The array and string callable syntaxes support has never been implemented. It would probably need first for PDepend to...
Phar can be directly downloaded via https://github.com/phpmd/phpmd/releases/download/2.12.0/phpmd.phar You can get them from https://github.com/phpmd/phpmd/releases in the "Assets" section from each release. This is our most up-to-date channel for both composer and...
Thanks. I did not create the PGP key `[email protected]` I'll will see for automation (old automation was on Travis, so we have to re-do it). Meanwhile the new key for...
Usage seems fair to me. The `public function getComment()` method would be a valid PR for me. About the null value, it requires a fix in PDepend.
Hi thanks, I like this feature a lot. I fixed the PHP 5.3 compatibility in your PR (please pull the change). Could you now add a unit test for that?...
@ravage84 A last word on this before merging?
Addressed https://github.com/phpmd/phpmd/pull/664/commits/8d296f83ed50e39c1fa796628a00c07f76f45061 If I well understand, this does not change the intent of the test.
Actually the test does not perform any assertions neither in PHP 7 because the `array_key_exists($name, $expected)` is always `false`.
I fixed the test to make it check what it intended to and sadly it fails (0 instead of 42 in test1 property).