Merge pull request #20 from DaveLiddament/bump/versions

BUMP Versions and add error identifiers

Author: DaveLiddament

.php-cs-fixer.php

@@ -23,6 +23,7 @@
             'phpdoc_align' => false,
             'phpdoc_to_comment' => false,
             'native_function_invocation' => false,
+            'nullable_type_declaration_for_default_null_value' => true,
         ]
     )
     ->setFinder($finder)

composer.json

@@ -1,18 +1,18 @@
 {
   "name": "dave-liddament/phpstan-php-language-extensions",
   "description": "PHPStan rules to implement the language extensions provided by the php-language-extensions",
-  "keywords": ["static analysis", "phpstan", "package attribute", "friend attribute"],
+  "keywords": ["static analysis", "phpstan", "namespace visibility attribute", "friend attribute"],
   "type": "phpstan-extension",
   "require": {
     "php": "~8.0.0 || ~8.1.0 || ~8.2.0 || ~8.3.0",
-    "phpstan/phpstan": "^1.10.34",
-    "dave-liddament/php-language-extensions": "^0.5.0 || ^0.6.0"
+    "phpstan/phpstan": "^1.11",
+    "dave-liddament/php-language-extensions": "^0.7.0"
   },
   "require-dev": {
     "phpunit/phpunit": "^9.6.12",
     "friendsofphp/php-cs-fixer": "^3.26.1",
     "php-parallel-lint/php-parallel-lint": "^1.3.2",
-    "dave-liddament/phpstan-rule-test-helper": "^0.1.0 || ^0.2.0"
+    "dave-liddament/phpstan-rule-test-helper": "^0.3.0"
   },
   "license": "MIT",
   "autoload": {

composer.lock

@@ -21,7 +21,7 @@ class TestConfig
 
     public function __construct(
         string $mode,
-        string $testNamespace = null,
+        ?string $testNamespace = null,
     ) {
         if (!in_array(
             needle: $mode,

src/Config/TestConfig.php

@@ -92,6 +92,6 @@ protected function getErrorOrNull(
             implode('|', $allowedCallingClasses)
         );
 
-        return RuleErrorBuilder::message($message)->build();
+        return RuleErrorBuilder::message($message)->identifier('phpExtensionLibrary.friend')->build();
     }
 }

src/Rules/AbstractFriendRule.php

@@ -108,7 +108,7 @@ protected function getErrorOrNull(
             $subNamespaces,
         );
 
-        return RuleErrorBuilder::message($message)->build();
+        return RuleErrorBuilder::message($message)->identifier('phpExtensionLibrary.namespace')->build();
     }
 
     /**

src/Rules/AbstractNamespaceVisibilityRule.php

@@ -86,6 +86,6 @@ protected function getErrorOrNull(
             $scope->getNamespace() ?? '<none>'
         );
 
-        return RuleErrorBuilder::message($message)->build();
+        return RuleErrorBuilder::message($message)->identifier('phpExtensionLibrary.package')->build();
     }
 }

src/Rules/AbstractPackageRule.php

@@ -71,6 +71,6 @@ protected function getErrorOrNull(
             $fullMethodName,
         );
 
-        return RuleErrorBuilder::message($message)->build();
+        return RuleErrorBuilder::message($message)->identifier('phpExtensionLibrary.testTag')->build();
     }
 }

src/Rules/AbstractTestTagRule.php

@@ -88,7 +88,7 @@ public function processNode(Node $node, Scope $scope): array
                         $classToUse
                     );
 
-                    $errors[] = RuleErrorBuilder::message($message)->build();
+                    $errors[] = RuleErrorBuilder::message($message)->identifier('phpExtensionLibrary.injectableVersion')->build();
                 }
             }
         }

src/Rules/InjectableVersionRule.php

@@ -47,7 +47,7 @@ public function processNode(Node $node, Scope $scope): array
             }
 
             $message = "Method {$methodName} has the Override attribute, but no matching parent method exists";
-            $errors[] = RuleErrorBuilder::message($message)->line($method->getLine())->build();
+            $errors[] = RuleErrorBuilder::message($message)->identifier('phpExtensionLibrary.override')->line($method->getLine())->build();
         }
 
         return $errors;