Skip to content

Commit

Permalink
Merge pull request #898 from ergebnis/fix/php-parser
Browse files Browse the repository at this point in the history
Fix: Remove `nikic/php-parser`
  • Loading branch information
localheinz authored Jan 8, 2025
2 parents 5b7b4c0 + 5474c3c commit f2867c9
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 16 deletions.
1 change: 0 additions & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@
"ergebnis/php-cs-fixer-config": "^6.39.0",
"ergebnis/phpunit-slow-test-detector": "^2.17.0",
"nette/di": "^3.1.10",
"nikic/php-parser": "^4.19.4",
"phpstan/extension-installer": "^1.4.3",
"phpstan/phpstan-deprecation-rules": "^2.0.1",
"phpstan/phpstan-phpunit": "^2.0.3",
Expand Down
22 changes: 12 additions & 10 deletions composer.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 0 additions & 4 deletions src/Classes/FinalRule.php
Original file line number Diff line number Diff line change
Expand Up @@ -134,10 +134,6 @@ private function hasWhitelistedAnnotation(Node\Stmt\Class_ $node): bool

$reformattedComment = $docComment->getReformattedText();

if (!\is_string($reformattedComment)) {
return false;
}

if (\is_int(\preg_match_all('/@(\S+)(?=\s|$)/', $reformattedComment, $matches))) {
foreach ($matches[1] as $annotation) {
foreach (self::$whitelistedAnnotations as $whitelistedAnnotation) {
Expand Down
1 change: 0 additions & 1 deletion src/Classes/PHPUnit/Framework/TestCaseWithSuffixRule.php
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,6 @@ public function processNode(
return [];
}

/** @var string $fullyQualifiedClassName */
$fullyQualifiedClassName = $node->namespacedName->toString();

$classReflection = $this->reflectionProvider->getClass($fullyQualifiedClassName);
Expand Down

0 comments on commit f2867c9

Please sign in to comment.