[config] Deprecate and remove @noRector annotation, as not reliable DX choice; use skip() method instead

[TomasVotruba]

This option was useful in the past, when Rector came with unreliable solutions and unknown types.

This was mostly caused by type-guessing from docblock in type declaration set. These rules are now removed, and are replaced by rules that work with reliable and safe strict types: https://twitter.com/rectorphp/status/1599001416718163968

• [TypeDeclaration] Disable and deprecate PropertyTypeDeclarationRector and TypedPropertyRector, to avoid breaks; use split strict rules instead #3122
• Deprecated type-declaration-strict to type-declaration set, as single group of rules #3103
• Remove deprecated ParamTypeDeclarationRector, that treated docs as strict types to keep type declaration set reliable #3111
• Deprecated and remove AddArrayParamDocTypeRector to work with strict type declarations only #3112

Saying that, these should no be ingnored anymore and fixed the the rule instead.

• It only works with FQN, which does not make sense in auto imported names
• The rename class could break it as well
• Docblocks are not much reliable 😕
• Lack of attribute form also suggest, this is not needed feature.
• It often hides a bug behind legal ignore
• The better option is often to ignore a pattern per rule, that is better achieved via config:

$rectorConfig->skip([
        \Rector\Privatization\Rector\Class_\FinalizeClassesWithoutChildrenRector::class => [
            '*/Entity/*'  
        ],
]);

If you still need to ignore some other rule for non-fixable reason, use skip() method instead, to keep it visible 👍

Currently, the Rector reports places in the code where @noRector is still used, so you can avoid unwanted changes. But in mostly cases, the ignore will not be needed.