Merge pull request #161 from stackkit/feature/PHP-8.4

Fix some PHPStan errors

Author: marickvantuil

phpstan.neon

@@ -5,6 +5,3 @@ parameters:
     paths:
         - src
     level: 9
-    checkMissingIterableValueType: false
-    ignoreErrors:
-        - '/Cannot call method when\(\) on mixed/'

src/TaskHandler.php

@@ -38,7 +38,7 @@ public function handle(?string $task = null): void
 
         // We want to catch any errors so we have more fine-grained control over
         // how tasks are retried. Cloud Tasks will retry the task if a 5xx status
-        // is returned. Because we manually manage retries by releaseing jobs,
+        // is returned. Because we manually manage retries by releasing jobs,
         // we never want to return a 5xx status as that will result in duplicate
         // job attempts.
         rescue(fn () => $this->run($task));

src/Worker.php

@@ -25,15 +25,15 @@
  */
 class Worker extends LaravelWorker
 {
-    public function process($connectionName, $job, WorkerOptions $options)
+    public function process($connectionName, $job, WorkerOptions $options): void
     {
         if ($this->supportsAsyncSignals()) {
             $this->listenForSignals();
 
             $this->registerTimeoutHandler($job, $options);
         }
 
-        return parent::process($connectionName, $job, $options);
+        parent::process($connectionName, $job, $options);
     }
 
     public function kill($status = 0, $options = null): void
@@ -43,10 +43,9 @@ public function kill($status = 0, $options = null): void
         // When running tests, we cannot run exit because it will kill the PHPunit process.
         // So, to still test that the application has exited, we will simply rely on the
         // WorkerStopped event that is fired when the worker is stopped.
-        if (app()->runningUnitTests()) {
-            return;
+        if (! app()->runningUnitTests()) {
+            exit($status);
         }
 
-        exit($status);
     }
 }