chore: cleanup (#97)

* chore: cs * fix phpstan errors * force psalm check at php 8.2 * add php version req in composer.json
rekalogika · May 30, 2024 · 38ac31b · 38ac31b
1 parent 021f882
commit 38ac31b
Show file tree

Hide file tree

Showing 7 changed files with 6 additions and 9 deletions.
diff --git a/composer.json b/composer.json
@@ -10,6 +10,7 @@
         }
     ],
     "require": {
+        "php": "^8.1",
         "doctrine/doctrine-bundle": "^2.11.3 || ^2.12",
         "doctrine/orm": "^2.16 || ^3.0",
         "doctrine/persistence": "^3.2",

diff --git a/packages/domain-event/src/DependencyInjection/CompilerPass/EntityManagerDecoratorPass.php b/packages/domain-event/src/DependencyInjection/CompilerPass/EntityManagerDecoratorPass.php
@@ -21,12 +21,12 @@
 
 /**
  * Decorates entity managers manually instead of relying on DI's decoration.
- * 
+ *
  * `ManagerRegistry::resetManager()` relies on the fact that entity managers are
  * lazy. However, when we decorate entity managers, the original entity manager
  * won't lazy anymore. This causes `resetManager()` to fail. the workaround is to
  * manually decorate entity managers.
- * 
+ *
  * @internal
  */
 final class EntityManagerDecoratorPass implements CompilerPassInterface

diff --git a/packages/domain-event/src/DependencyInjection/CompilerPass/ProfilerWorkaroundPass.php b/packages/domain-event/src/DependencyInjection/CompilerPass/ProfilerWorkaroundPass.php
@@ -28,9 +28,9 @@
  * accepts "Doctrine\Bundle\DoctrineBundle\Registry",
  * "Rekalogika\DomainEvent\Doctrine\DomainEventAwareManagerRegistryImplementation"
  * passed.
- * 
+ *
  * fix in upstream: https://github.com/doctrine/DoctrineBundle/pull/1764
- * 
+ *
  * @todo remove after we bump to doctrine/doctrine-bundle 2.12
  *
  * @internal

diff --git a/tests/Framework/EventListener/BookEventEventBusListener.php b/tests/Framework/EventListener/BookEventEventBusListener.php
@@ -53,7 +53,6 @@ public function onRemoveCalled(): bool
         return $this->onRemoveCalled;
     }
 
-    /** @phpstan-impure */
     public function onChangeCalled(): int
     {
         return $this->onChangeCalled;

diff --git a/tests/Framework/EventListener/BookEventPostFlushListener.php b/tests/Framework/EventListener/BookEventPostFlushListener.php
@@ -52,7 +52,6 @@ public function onRemoveCalled(): bool
         return $this->onRemoveCalled;
     }
 
-    /** @phpstan-impure */
     public function onChangeCalled(): int
     {
         return $this->onChangeCalled;

diff --git a/tests/Framework/EventListener/BookEventPreFlushListener.php b/tests/Framework/EventListener/BookEventPreFlushListener.php
@@ -52,7 +52,6 @@ public function onRemoveCalled(): bool
         return $this->onRemoveCalled;
     }
 
-    /** @phpstan-impure */
     public function onChangeCalled(): int
     {
         return $this->onChangeCalled;

diff --git a/tests/Framework/EventListener/PostEventEventBusListener.php b/tests/Framework/EventListener/PostEventEventBusListener.php
@@ -52,8 +52,7 @@ public function onRemoveCalled(): bool
     {
         return $this->onRemoveCalled;
     }
-
-    /** @phpstan-impure */
+
     public function onChangeCalled(): int
     {
         return $this->onChangeCalled;