Fix styling

Author: freekmurze

src/BackupDestination/BackupCollection.php

@@ -13,7 +13,7 @@ class BackupCollection extends Collection
     public static function createFromFiles(?FileSystem $disk, array $files): self
     {
         return (new static($files))
-            ->filter(fn (string $path) => (new File())->isZipFile($disk, $path))
+            ->filter(fn (string $path) => (new File)->isZipFile($disk, $path))
             ->map(fn (string $path) => new Backup($disk, $path))
             ->sortByDesc(fn (Backup $backup) => $backup->date()->timestamp)
             ->values();

src/BackupDestination/BackupDestination.php

@@ -20,7 +20,7 @@ class BackupDestination
 
     protected ?BackupCollection $backupCollectionCache = null;
 
-    public function __construct(Filesystem $disk = null, string $backupName, string $diskName)
+    public function __construct(?Filesystem $disk, string $backupName, string $diskName)
     {
         $this->disk = $disk;
 

src/BackupServiceProvider.php

@@ -56,7 +56,7 @@ protected function registerDiscordChannel()
     {
         Notification::resolved(function (ChannelManager $service) {
             $service->extend('discord', function ($app) {
-                return new DiscordChannel();
+                return new DiscordChannel;
             });
         });
     }

src/Commands/ListCommand.php

@@ -20,7 +20,7 @@ class ListCommand extends BaseCommand
     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.");
+            $this->warn('Warning! Your config file still uses the old monitorBackups key. Update it to monitor_backups.');
         }
 
         $statuses = BackupDestinationStatusFactory::createForMonitorConfig(config('backup.monitor_backups'));
@@ -39,8 +39,8 @@ protected function displayOverview(Collection $backupDestinationStatuses)
         });
 
         $this->table($headers, $rows, 'default', [
-            4 => new RightAlignedTableStyle(),
-            6 => new RightAlignedTableStyle(),
+            4 => new RightAlignedTableStyle,
+            6 => new RightAlignedTableStyle,
         ]);
 
         return $this;
@@ -98,7 +98,7 @@ protected function displayFailures(Collection $backupDestinationStatuses)
         return $this;
     }
 
-    protected function getFormattedBackupDate(Backup $backup = null)
+    protected function getFormattedBackupDate(?Backup $backup = null)
     {
         return is_null($backup)
             ? 'No backups present'

src/Commands/MonitorCommand.php

@@ -18,7 +18,7 @@ class MonitorCommand extends BaseCommand implements Isolatable
     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.");
+            $this->warn('Warning! Your config file still uses the old monitorBackups key. Update it to monitor_backups.');
         }
 
         $hasError = false;

src/Events/BackupHasFailed.php

@@ -10,6 +10,5 @@ class BackupHasFailed
     public function __construct(
         public Exception $exception,
         public ?BackupDestination $backupDestination = null,
-    ) {
-    }
+    ) {}
 }

src/Events/BackupManifestWasCreated.php

@@ -8,6 +8,5 @@ class BackupManifestWasCreated
 {
     public function __construct(
         public Manifest $manifest,
-    ) {
-    }
+    ) {}
 }

src/Events/BackupWasSuccessful.php

@@ -8,6 +8,5 @@ class BackupWasSuccessful
 {
     public function __construct(
         public BackupDestination $backupDestination,
-    ) {
-    }
+    ) {}
 }

src/Events/BackupZipWasCreated.php

@@ -6,6 +6,5 @@ class BackupZipWasCreated
 {
     public function __construct(
         public string $pathToZip,
-    ) {
-    }
+    ) {}
 }

src/Events/CleanupHasFailed.php

@@ -10,6 +10,5 @@ class CleanupHasFailed
     public function __construct(
         public Exception $exception,
         public ?BackupDestination $backupDestination = null,
-    ) {
-    }
+    ) {}
 }