fix: drop PreventParentMethodVisibilityOverrideRule ? (fixes symplify/phpstan-rules#227)

README.md

@@ -903,55 +903,6 @@ class SomeClass
 
 <br>
 
-### PreventParentMethodVisibilityOverrideRule
-
-Change `"%s()"` method visibility to "%s" to respect parent method visibility.
-
-```yaml
-rules:
-    - Symplify\PHPStanRules\Rules\PreventParentMethodVisibilityOverrideRule
-```
-
-```php
-class SomeParentClass
-{
-    public function run()
-    {
-    }
-}
-
-class SomeClass extends SomeParentClass
-{
-    protected function run()
-    {
-    }
-}
-```
-
-:x:
-
-<br>
-
-```php
-class SomeParentClass
-{
-    public function run()
-    {
-    }
-}
-
-class SomeClass extends SomeParentClass
-{
-    public function run()
-    {
-    }
-}
-```
-
-:+1:
-
-<br>
-
 ### RequiredOnlyInAbstractRule
 
 `@required` annotation should be used only in abstract classes, to child classes can use clean `__construct()` service injection.

config/static-rules.neon

@@ -9,7 +9,6 @@ rules:
     - Symplify\PHPStanRules\Rules\RequireAttributeNameRule
 
     - Symplify\PHPStanRules\Rules\Enum\RequireUniqueEnumConstantRule
-    - Symplify\PHPStanRules\Rules\PreventParentMethodVisibilityOverrideRule
 
     - Symplify\PHPStanRules\Rules\ForbiddenMultipleClassLikeInOneFileRule
     - Symplify\PHPStanRules\Rules\NoReferenceRule

src/Enum/RuleIdentifier.php

@@ -14,8 +14,6 @@ final class RuleIdentifier
 
     public const string PHP_PARSER_NO_LEADING_BACKSLASH_IN_NAME = 'phpParser.noLeadingBackslashInName';
 
-    public const string PARENT_METHOD_VISIBILITY_OVERRIDE = 'symplify.parentMethodVisibilityOverride';
-
     public const string NO_RETURN_SETTER_METHOD = 'symplify.noReturnSetterMethod';
 
     public const string FORBIDDEN_STATIC_CLASS_CONST_FETCH = 'symplify.forbiddenStaticClassConstFetch';