drop 8.1

spatie · Jun 11, 2024 · f4067e0 · f4067e0
1 parent 1513982
commit f4067e0
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml
@@ -17,7 +17,7 @@ jobs:
             fail-fast: false
             matrix:
                 os: [ubuntu-latest]
-                php: [8.3, 8.2, 8.1]
+                php: [8.3, 8.2]
                 laravel: [11.*, 10.*]
                 stability: [prefer-lowest, prefer-stable]
                 carbon: [^2.63]
@@ -26,9 +26,6 @@ jobs:
                       testbench: 8.*
                     - laravel: 11.*
                       testbench: 9.*
-                exclude:
-                    - laravel: 11.*
-                      php: 8.1
 
         name: P${{ matrix.php }} - L${{ matrix.laravel }} - ${{ matrix.stability }} - ${{ matrix.os }}
 

diff --git a/src/Config/BackupConfig.php b/src/Config/BackupConfig.php
@@ -2,13 +2,11 @@
 
 namespace Spatie\Backup\Config;
 
+use Spatie\Backup\Exceptions\InvalidConfig;
 use Spatie\Backup\Support\Data;
 
 class BackupConfig extends Data
 {
-    /**
-     * @param  positive-int  $tries
-     */
     protected function __construct(
         public string $name,
         public SourceConfig $source,
@@ -24,6 +22,9 @@ protected function __construct(
         public int $retryDelay,
         public ?MonitoredBackupsConfig $monitoredBackups,
     ) {
+        if ($this->tries < 1) {
+            throw InvalidConfig::integerMustBePositive('tries');
+        }
     }
 
     /** @param array<mixed> $data */

diff --git a/src/Exceptions/InvalidConfig.php b/src/Exceptions/InvalidConfig.php
@@ -15,4 +15,9 @@ public static function missingSender(): static
     {
         return new static('No sender email address specified.');
     }
+
+    public static function integerMustBePositive(string $name): static
+    {
+        return new static("`{$name}` must be a positive number.");
+    }
 }