Kyle
Kyle
Fix proposal submitted as #1068
I approve your proposal 2. and it seems perfectly fine to me to keep `minimum` naming for `ShortMethodName`, `ShortVariable` > change the parameters name in minor versions, breaking our checks...
Revisiting this, as I'm trying to imagine what the common base for Rule could be in v3. I like the idea of `VariableName` (or some verbose `LengthOfVariableName`) that would check...
null, default and new were fixed in [2.13.0](https://github.com/pdepend/pdepend/releases/tag/2.13.0) And the rest of the keyword is fixed on `pdepend/pdepend:dev-master`
Yes, but sadly many of them are incomplete, for instance, many of our PHPDoc types miss a nullable flag: `getReport()` which had `@return Report` but actually `@return Report|null` so it...
And why the test fail is actually for the same reason. PHPDoc said `@param int` for `getIntProperty` but it's actually `int|null` and it rely on the `null` to throw or...
Oh, I have already few other branches soon to be pushed. 😄
For many rules, we check only the symbols of the files (functions/classes/traits). Code outside of symbols is not analyzed by the parser. We sure could improve this to handle procedural...
This is probably related to the same root cause than: #540 #573
This should be a dedicated rule I guess, because it means it's an exception for CamelCaseParameterName, CamelCaseVariableName and UnusedVariableRule, then it would in fact be mandatory for this variable not...