Merge pull request #493 from greg0ire/remove-psalm

Drop Psalm in favor of just PHPStan

.gitattributes

@@ -12,5 +12,3 @@
 /phpcs.xml.dist export-ignore
 /phpstan.neon.dist export-ignore
 /phpstan-baseline.neon export-ignore
-/psalm.xml export-ignore
-/psalm-baseline.xml export-ignore

.github/workflows/static-analysis.yml

@@ -11,4 +11,4 @@ on:
 jobs:
   static-analysis:
     name: "Static Analysis"
-    uses: "doctrine/.github/.github/workflows/static-analysis[email protected]"
+    uses: "doctrine/.github/.github/workflows/phpstan[email protected]"

composer.json

@@ -33,8 +33,7 @@
         "phpstan/phpstan": "^1.10",
         "phpunit/phpunit": "^9.6.13 || ^10.4.2",
         "symfony/cache": "^5.4 || ^6.3 || ^7",
-        "symfony/var-exporter": "^5.4 || ^6.3 || ^7",
-        "vimeo/psalm": "^5.9"
+        "symfony/var-exporter": "^5.4 || ^6.3 || ^7"
     },
     "suggest": {
         "alcaeus/mongo-php-adapter": "For using MongoDB ODM 1.3 with PHP 7 (deprecated)",

src/AbstractFixture.php

@@ -82,10 +82,10 @@ public function addReference($name, $object)
      * @see ReferenceRepository::getReference()
      *
      * @param string $name
-     * @psalm-param class-string<T>|null $class
+     * @phpstan-param class-string<T>|null $class
      *
      * @return object
-     * @psalm-return ($class is null ? object : T)
+     * @phpstan-return ($class is null ? object : T)
      *
      * @template T of object
      */
@@ -110,7 +110,7 @@ public function getReference($name, ?string $class = null)
      * @see ReferenceRepository::hasReference()
      *
      * @param string $name
-     * @psalm-param class-string $class
+     * @phpstan-param class-string $class
      *
      * @return bool
      */

src/DependentFixtureInterface.php

@@ -13,7 +13,7 @@ interface DependentFixtureInterface
      * This method must return an array of fixtures classes
      * on which the implementing class depends on
      *
-     * @psalm-return array<class-string<FixtureInterface>>
+     * @phpstan-return array<class-string<FixtureInterface>>
      */
     public function getDependencies();
 }

src/Loader.php

@@ -40,14 +40,14 @@ class Loader
     /**
      * Array of fixture object instances to execute.
      *
-     * @psalm-var array<class-string<FixtureInterface>, FixtureInterface>
+     * @phpstan-var array<class-string<FixtureInterface>, FixtureInterface>
      */
     private array $fixtures = [];
 
     /**
      * Array of ordered fixture object instances.
      *
-     * @psalm-var array<class-string<FixtureInterface>|int, FixtureInterface>
+     * @phpstan-var array<class-string<FixtureInterface>|int, FixtureInterface>
      */
     private array $orderedFixtures = [];
 
@@ -175,7 +175,7 @@ public function addFixture(FixtureInterface $fixture)
     /**
      * Returns the array of data fixtures to execute.
      *
-     * @psalm-return array<class-string<FixtureInterface>|int, FixtureInterface>
+     * @phpstan-return array<class-string<FixtureInterface>|int, FixtureInterface>
      */
     public function getFixtures()
     {
@@ -200,7 +200,7 @@ public function getFixtures()
      * Check if a given fixture is transient and should not be considered a data fixtures
      * class.
      *
-     * @psalm-param class-string<object> $className
+     * @phpstan-param class-string<object> $className
      *
      * @return bool
      */
@@ -264,7 +264,7 @@ private function orderFixturesByNumber(): void
      */
     private function orderFixturesByDependencies()
     {
-        /** @psalm-var array<class-string<DependentFixtureInterface>, int> */
+        /** @phpstan-var array<class-string<DependentFixtureInterface>, int> */
         $sequenceForClasses = [];
 
         // If fixtures were already ordered by number then we need
@@ -365,7 +365,7 @@ private function orderFixturesByDependencies()
         $this->orderedFixtures = array_merge($this->orderedFixtures, $orderedFixtures);
     }
 
-    /** @psalm-param iterable<class-string> $dependenciesClasses */
+    /** @phpstan-param iterable<class-string> $dependenciesClasses */
     private function validateDependencies(iterable $dependenciesClasses): bool
     {
         $loadedFixtureClasses = array_keys($this->fixtures);
@@ -383,10 +383,10 @@ private function validateDependencies(iterable $dependenciesClasses): bool
     }
 
     /**
-     * @psalm-param array<class-string<DependentFixtureInterface>, int> $sequences
-     * @psalm-param iterable<class-string<FixtureInterface>>|null       $classes
+     * @phpstan-param array<class-string<DependentFixtureInterface>, int> $sequences
+     * @phpstan-param iterable<class-string<FixtureInterface>>|null       $classes
      *
-     * @psalm-return array<class-string<FixtureInterface>>
+     * @phpstan-return array<class-string<FixtureInterface>>
      */
     private function getUnsequencedClasses(array $sequences, ?iterable $classes = null): array
     {
@@ -410,10 +410,10 @@ private function getUnsequencedClasses(array $sequences, ?iterable $classes = nu
     /**
      * Load fixtures from files contained in iterator.
      *
-     * @psalm-param Iterator<SplFileInfo> $iterator Iterator over files from
+     * @phpstan-param Iterator<SplFileInfo> $iterator Iterator over files from
      *                                              which fixtures should be loaded.
      *
-     * @psalm-return list<FixtureInterface> $fixtures Array of loaded fixture object instances.
+     * @phpstan-return list<FixtureInterface> $fixtures Array of loaded fixture object instances.
      */
     private function loadFromIterator(Iterator $iterator): array
     {

src/ReferenceRepository.php

@@ -27,15 +27,15 @@ class ReferenceRepository
      * List of named references to the fixture objects
      * gathered during fixure loading
      *
-     * @psalm-var array<string, object>
+     * @phpstan-var array<string, object>
      */
     private array $references = [];
 
     /**
      * List of named references to the fixture objects
      * gathered during fixure loading
      *
-     * @psalm-var array<class-string, array<string, object>>
+     * @phpstan-var array<class-string, array<string, object>>
      */
     private array $referencesByClass = [];
 
@@ -44,7 +44,7 @@ class ReferenceRepository
      * in case a reference gets no longer managed, it will
      * use a proxy referenced by this identity
      *
-     * @psalm-var array<string, mixed>
+     * @phpstan-var array<string, mixed>
      */
     private array $identities = [];
 
@@ -53,7 +53,7 @@ class ReferenceRepository
      * in case a reference gets no longer managed, it will
      * use a proxy referenced by this identity
      *
-     * @psalm-var array<class-string, array<string, mixed>>
+     * @phpstan-var array<class-string, array<string, mixed>>
      */
     private array $identitiesByClass = [];
 
@@ -200,10 +200,10 @@ public function addReference($name, $object)
      * named by $name
      *
      * @param string $name
-     * @psalm-param class-string<T>|null $class
+     * @phpstan-param class-string<T>|null $class
      *
      * @return object
-     * @psalm-return ($class is null ? object : T)
+     * @phpstan-return ($class is null ? object : T)
      *
      * @throws OutOfBoundsException - if repository does not exist.
      *
@@ -257,7 +257,7 @@ public function getReference($name, ?string $class = null)
      * named by $name
      *
      * @param string $name
-     * @psalm-param class-string $class
+     * @phpstan-param class-string $class
      *
      * @return bool
      */
@@ -324,7 +324,7 @@ public function hasIdentity($name, ?string $class = null)
      *
      * Get all stored identities
      *
-     * @psalm-return array<string, object>
+     * @phpstan-return array<string, object>
      */
     public function getIdentities()
     {
@@ -334,7 +334,7 @@ public function getIdentities()
     /**
      * Get all stored identities
      *
-     * @psalm-return array<class-string, array<string, object>>
+     * @phpstan-return array<class-string, array<string, object>>
      */
     public function getIdentitiesByClass(): array
     {
@@ -346,7 +346,7 @@ public function getIdentitiesByClass(): array
      *
      * Get all stored references
      *
-     * @psalm-return array<string, object>
+     * @phpstan-return array<string, object>
      */
     public function getReferences()
     {
@@ -356,7 +356,7 @@ public function getReferences()
     /**
      * Get all stored references
      *
-     * @psalm-return array<class-string, array<string, object>>
+     * @phpstan-return array<class-string, array<string, object>>
      */
     public function getReferencesByClass(): array
     {

src/Sorter/Vertex.php

@@ -18,7 +18,7 @@ class Vertex
     public const IN_PROGRESS = 1;
     public const VISITED     = 2;
 
-    /** @psalm-var self::* */
+    /** @phpstan-var self::* */
     public int $state = self::NOT_VISITED;
 
     /** Actual node value. */