[Cleanup] Remove @changelog as no longer maintained/used, use RuleDefinition instead#6035
[Cleanup] Remove @changelog as no longer maintained/used, use RuleDefinition instead#6035TomasVotruba merged 4 commits intomainfrom
Conversation
| use Webmozart\Assert\Assert; | ||
|
|
||
| /** | ||
| * @changelog https://php.watch/versions/8.1/version_compare-operator-restrictions |
There was a problem hiding this comment.
I think @changelog can be replaced with @see
There was a problem hiding this comment.
That's an option. Should be part of rule definition though, otherwise it's only visible when rule is opened (rarely :))
There was a problem hiding this comment.
E.g. this link is completelly wrong. It's a different feature.
That's why I want to remove those, as docblocks are not part of rule definition and can be incorrectly copy-pasted.
There was a problem hiding this comment.
Just checked couple of those and they're either:
- link to 3v4l.org = those should be in a code example snippet or tests
- links to PHP rfc
- dead links
I'll keep them removed, as I want to keep focused on sets rather than particular rules. Similar way PHPStan does, to make it easy to install Rector and run it 👍
79472f8 to
217e348
Compare
| , | ||
| [ | ||
| AddLiteralSeparatorToNumberRector::LIMIT_VALUE => 1_000_000, | ||
| self::LIMIT_VALUE => 1_000_000, |
There was a problem hiding this comment.
configurable example should use original class reference instead of self:: to ease copy paste config
We've added this feature many years ago, but we no longer maintain, nor use it. There is already rule RuleDefinition that should be used for any explanation.
Time to cleanup the code 👍