Merge 607137f into 9fa3cb5

php-coveralls · May 4, 2023 · 37f3c3c · 37f3c3c
2 parents 9fa3cb5 + 607137f
commit 37f3c3c
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 6 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -36,13 +36,29 @@ jobs:
                         os: ubuntu-latest
                     -   php: '7.4'
                         os: ubuntu-latest
-                    -   php: '7.4'
-                        os: windows-latest
-                        coverage: 'pcov'
                     -   php: '8.0'
                         os: ubuntu-latest
                         symfony: '^6.0'
+                    -   php: '8.1'
+                        os: ubuntu-latest
+                        symfony: '^6.0'
+                    -   php: '8.2'
+                        os: ubuntu-latest
+                        symfony: '^6.0'
                         coverage: 'pcov'
+                    -   php: '8.2'
+                        os: ubuntu-latest
+                        symfony: '^6.0'
+                        coverage: 'xdebug'
+                    -   php: '8.2'
+                        os: windows-latest
+                        symfony: '^6.0'
+                        coverage: 'pcov'
+                    -   php: '8.3'
+                        os: ubuntu-latest
+                        symfony: '^6.0'
+
+        continue-on-error: ${{ matrix.php == '8.3' }}
 
         steps:
             -   name: Checkout source
@@ -79,11 +95,11 @@ jobs:
                 run: composer config extra.symfony.require ${{ matrix.symfony }}
 
             -   name: Update composer dependencies
-                if: matrix.php != '8.2'
+                if: matrix.php != '8.3'
                 run: composer update -o --no-interaction --no-progress ${{ matrix.composer-flags }}
 
             -   name: "Update composer dependencies (PHP 8.2, ignore platform)"
-                if: matrix.php == '8.2'
+                if: matrix.php == '8.3'
                 run: composer update -o --no-interaction --no-progress ${{ matrix.composer-flags }} --ignore-platform-req=php+
 
             -   name: Run test suite
@@ -164,6 +180,9 @@ jobs:
                     -   php: '5.5'
                     -   php: '7.2'
                     -   php: '8.0'
+                    -   php: '8.3'
+
+        continue-on-error: ${{ matrix.php == '8.3' }}
 
         steps:
             -   name: Checkout

diff --git a/src/Bundle/CoverallsBundle/Repository/JobsRepository.php b/src/Bundle/CoverallsBundle/Repository/JobsRepository.php
@@ -155,7 +155,10 @@ protected function dumpJsonFile()
 
         $this->api->dumpJsonFile();
 
-        $filesize = number_format(filesize($jsonPath) / 1024, 2); // kB
+        $filesize = 0;
+        if (\is_string($jsonPath) && file_exists($jsonPath)) {
+            $filesize = number_format(filesize($jsonPath) / 1024, 2); // kB
+        }
         $this->logger->info(sprintf('File size: <info>%s</info> kB', $filesize));
 
         return $this;