Add RemoveUselessTypeInParamTagRector rule
#5811
Conversation
| + * @param $a | ||
| + * @param $b description |
There was a problem hiding this comment.
this is wrong, and param tag become mixed, and there is already RemoveUselessParamTagRector to remove useless param tag https://github.com/rectorphp/rector-src/blob/main/build/target-repository/docs/rector_rules_overview.md#removeuselessparamtagrector for it
There was a problem hiding this comment.
The param doesn't become mixed ; the type is optional, as stated here. It can be omitted and avoid duplicating info. I still think this could be a great addition. The goal here is not to remove useless param, it removes useless type in @param when it is already present in the method signature 🙂
There was a problem hiding this comment.
you can create your own custom rule for that.
|
I am closing it :), see the reasoning #5811 (review), you can create your own custom rule for your usecase. |
As discussed here symfony/symfony#54489 (comment), this rule would help achieve this in the whole codebase. That would be great to see this rule bundled in Rector directly 🙂