Merge pull request #224 from erikn69/patch-1

PHP 8.4 tests
spatie · Nov 4, 2024 · 4fd2203 · 4fd2203
2 parents d873dcf + b11bf8d
commit 4fd2203
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -9,7 +9,7 @@ jobs:
             fail-fast: false
             matrix:
                 os: [ubuntu-latest]
-                php: [8.3, 8.2, 8.1, 8.0, 7.4, 7.3]
+                php: [8.4, 8.3, 8.2, 8.1, 8.0, 7.4, 7.3]
                 dependency-version: [prefer-lowest, prefer-stable]
 
         name: P${{ matrix.php }} - ${{ matrix.dependency-version }} - ${{ matrix.os }}

diff --git a/src/OptimizerChain.php b/src/OptimizerChain.php
@@ -62,11 +62,11 @@ public function useLogger(LoggerInterface $log)
         return $this;
     }
 
-    public function optimize(string $pathToImage, string $pathToOutput = null)
+    public function optimize(string $pathToImage, ?string $pathToOutput = null)
     {
         if ($pathToOutput) {
             $check = copy($pathToImage, $pathToOutput);
-            if($check == false) {
+            if ($check == false) {
                 throw new InvalidArgumentException("Cannot copy file");
             }
             $pathToImage = $pathToOutput;