The Easiest way to add coding standard to your PHP project
PHPStan rules to measure cognitive complexity of your classes and methods
[automated] Update Rector Dependencies (#1201)
Co-authored-by: TomasVotruba TomasVotruba@users.noreply.github.com
Automated changes by create-pull-request GitHub action
Automated changes by create-pull-request GitHub action
Updated Rector to commit 34ff3246d3d5f040ca66f30a1eb883923962aecd
https://github.com/rectorphp/rector-src/commit/34ff3246d3d5f040ca66f30a1eb883923962aecd [CodingStyle] Remove parent attribute usage on RemoveFinalFromConstRector (#3536)
Updated Rector to commit 0d5169dd77c909421aa1de31aac7bdcd502d66aa
https://github.com/rectorphp/rector-src/commit/0d5169dd77c909421aa1de31aac7bdcd502d66aa [Php70] Remove previous attribute usage on ReduceMultipleDefaultSwitchRector (#3537)
Updated Rector to commit 180d9a4fad89124d3f547af178f5179e4d4277ee
https://github.com/rectorphp/rector-src/commit/180d9a4fad89124d3f547af178f5179e4d4277ee [DeadCode] Remove previous attribute usage on RemoveDeadConditionAboveReturnRector (#3538)
Both are fine 👍
Not sure why, maybe draft is blocker
[CodingStyle] Remove parent attribute usage on RemoveFinalFromConstRector (#3536)
CodingStyle] Remove parent attribute usage on RemoveFinalFromConstRector
[ci-review] Rector Rectify
Co-authored-by: GitHub Action actions@github.com
[Php70] Remove previous attribute usage on ReduceMultipleDefaultSwitchRector (#3537)
Ref https://github.com/rectorphp/rector/issues/7849
Ref https://github.com/rectorphp/rector/issues/7849
Thanks
[DeadCode] Remove previous attribute usage on RemoveDeadConditionAboveReturnRector (#3538)
[DeadCode] Remove previous attribute usage on RemoveDeadConditionAboveReturnRector
phpstan
[ci-review] Rector Rectify
Fix static calls on variables in RequireThisOnParentMethodCallRule (#19)
Thanks :+1:
The RequireThisOnParentMethodCallRule
was overly strict and reported all static calls on variables as well.
In most cases you should never make a static call on a variable (the only valid case would be making a static call on a variable that contains a class string), but that is not the scope of this rule.
Now the RequireThisOnParentMethodCallRule
skips all static calls on variables
We used to have such backlog, but it attracted passer-by developers who came with idea, dropped it and never integrated the rule.
Instead, for the long-term project it's healthier to attract contributors, who open PR before creating issue and keep on improving the project :+1:
We'll need a rebase here.
This might be a blocker for many people. What is the added value here?