return Command constant instead of integer

src/Commands/BackupCommand.php

@@ -16,7 +16,7 @@ class BackupCommand extends BaseCommand
 
     protected $description = 'Run the backup.';
 
-    public function handle()
+    public function handle(): int
     {
         consoleOutput()->comment($this->currentTry > 1 ? sprintf('Attempt n°%d...', $this->currentTry) : 'Starting backup...');
 
@@ -63,6 +63,8 @@ public function handle()
             $backupJob->run();
 
             consoleOutput()->comment('Backup completed!');
+
+            return static::SUCCESS;
         } catch (Exception $exception) {
             if ($this->shouldRetry()) {
                 if ($this->hasRetryDelay('backup')) {
@@ -82,7 +84,7 @@ public function handle()
                 event(new BackupHasFailed($exception));
             }
 
-            return 1;
+            return static::FAILURE;
         }
     }
 

src/Commands/CleanupCommand.php

@@ -28,7 +28,7 @@ public function __construct(CleanupStrategy $strategy)
         $this->strategy = $strategy;
     }
 
-    public function handle()
+    public function handle(): int
     {
         consoleOutput()->comment($this->currentTry > 1 ? sprintf('Attempt n°%d...', $this->currentTry) : 'Starting cleanup...');
 
@@ -46,6 +46,8 @@ public function handle()
             $cleanupJob->run();
 
             consoleOutput()->comment('Cleanup completed!');
+
+            return static::SUCCESS;
         } catch (Exception $exception) {
             if ($this->shouldRetry()) {
                 if ($this->hasRetryDelay('cleanup')) {
@@ -61,7 +63,7 @@ public function handle()
                 event(new CleanupHasFailed($exception));
             }
 
-            return 1;
+            return static::FAILURE;
         }
     }
 }

src/Commands/ListCommand.php

@@ -17,7 +17,7 @@ class ListCommand extends BaseCommand
     /** @var string */
     protected $description = 'Display a list of all backups.';
 
-    public function handle()
+    public function handle(): int
     {
         if (config()->has('backup.monitorBackups')) {
             $this->warn("Warning! Your config file still uses the old monitorBackups key. Update it to monitor_backups.");
@@ -26,6 +26,8 @@ public function handle()
         $statuses = BackupDestinationStatusFactory::createForMonitorConfig(config('backup.monitor_backups'));
 
         $this->displayOverview($statuses)->displayFailures($statuses);
+
+        return static::SUCCESS;
     }
 
     protected function displayOverview(Collection $backupDestinationStatuses)

src/Commands/MonitorCommand.php

@@ -14,7 +14,7 @@ class MonitorCommand extends BaseCommand
     /** @var string */
     protected $description = 'Monitor the health of all backups.';
 
-    public function handle()
+    public function handle(): int
     {
         if (config()->has('backup.monitorBackups')) {
             $this->warn("Warning! Your config file still uses the old monitorBackups key. Update it to monitor_backups.");
@@ -39,7 +39,9 @@ public function handle()
         }
 
         if ($hasError) {
-            return 1;
+            return static::FAILURE;
         }
+
+        return static::SUCCESS;
     }
 }