bug fixes

SonsOfPHP · Aug 1, 2024 · 17bc970 · 17bc970
1 parent 3a6650b
commit 17bc970
Show file tree

Hide file tree

Showing 9 changed files with 9 additions and 21 deletions.
diff --git a/src/SonsOfPHP/Bard/src/Console/Command/CopyCommand.php b/src/SonsOfPHP/Bard/src/Console/Command/CopyCommand.php
@@ -18,11 +18,10 @@
  */
 final class CopyCommand extends AbstractCommand
 {
-    protected static $defaultName = 'copy';
-
     protected function configure(): void
     {
         $this
+            ->setName('copy')
             ->setDescription('Copies a file to each package')
             ->addOption('dry-run', null, InputOption::VALUE_NONE, 'Dry Run (Do not make any changes)')
             ->addArgument('source', InputArgument::REQUIRED, 'Source file to copy')

diff --git a/src/SonsOfPHP/Bard/src/Console/Command/InitCommand.php b/src/SonsOfPHP/Bard/src/Console/Command/InitCommand.php
@@ -16,11 +16,10 @@
  */
 final class InitCommand extends AbstractCommand
 {
-    protected static $defaultName = 'init';
-
     protected function configure(): void
     {
         $this
+            ->setName('init')
             ->setDescription('Creates the initial bard.json file')
         ;
     }

diff --git a/src/SonsOfPHP/Bard/src/Console/Command/InstallCommand.php b/src/SonsOfPHP/Bard/src/Console/Command/InstallCommand.php
@@ -14,16 +14,10 @@
  */
 final class InstallCommand extends AbstractCommand
 {
-    protected static $defaultName = 'install';
-
-    // public function __construct()
-    // {
-    //    parent::__construct();
-    // }
-
     protected function configure(): void
     {
         $this
+            ->setName('install')
             ->setDescription('Runs composer install for all packages')
         ;
     }

diff --git a/src/SonsOfPHP/Bard/src/Console/Command/MergeCommand.php b/src/SonsOfPHP/Bard/src/Console/Command/MergeCommand.php
@@ -31,7 +31,6 @@
  */
 final class MergeCommand extends AbstractCommand
 {
-    protected static $defaultName = 'merge';
     private readonly Json $json;
     private array $bardConfig;
     private string $mainComposerFile;
@@ -47,6 +46,7 @@ public function __construct()
     protected function configure(): void
     {
         $this
+            ->setName('merge')
             ->setDescription('Merges package composer.json files into main composer.json file')
             ->addOption('dry-run', null, InputOption::VALUE_NONE, 'Dry Run (Do not make any changes)')
             ->addArgument('package', InputArgument::OPTIONAL, 'Which package?')

diff --git a/src/SonsOfPHP/Bard/src/Console/Command/PullCommand.php b/src/SonsOfPHP/Bard/src/Console/Command/PullCommand.php
@@ -16,11 +16,10 @@
  */
 final class PullCommand extends AbstractCommand
 {
-    protected static $defaultName = 'pull';
-
     protected function configure(): void
     {
         $this
+            ->setName('pull')
             ->setDescription('Pull changes from package repos')
             ->addOption('branch', null, InputOption::VALUE_REQUIRED, 'What branch we working with?', 'main')
         ;

diff --git a/src/SonsOfPHP/Bard/src/Console/Command/PushCommand.php b/src/SonsOfPHP/Bard/src/Console/Command/PushCommand.php
@@ -19,11 +19,10 @@
  */
 final class PushCommand extends AbstractCommand
 {
-    protected static $defaultName = 'push';
-
     protected function configure(): void
     {
         $this
+            ->setName('push')
             ->setDescription('Push changes to package repos using git subtree push')
             ->addOption('branch', null, InputOption::VALUE_REQUIRED, 'What branch we working with?', 'main')
             ->addOption('dry-run', null, InputOption::VALUE_NONE, 'Dry Run (Do not make any changes)')

diff --git a/src/SonsOfPHP/Bard/src/Console/Command/ReleaseCommand.php b/src/SonsOfPHP/Bard/src/Console/Command/ReleaseCommand.php
@@ -25,14 +25,14 @@
  */
 final class ReleaseCommand extends AbstractCommand
 {
-    protected static $defaultName = 'release';
     private JsonFile $bardConfig;
     private VersionInterface|Version|null $releaseVersion = null;
     private bool $isDryRun = true;
 
     protected function configure(): void
     {
         $this
+            ->setName('release')
             ->setDescription('Create a new release')
             ->addOption('dry-run', null, InputOption::VALUE_NONE, 'Execute a dry-run with nothing being updated or released')
             ->addOption('branch', null, InputOption::VALUE_REQUIRED, 'What branch we working with?', 'main')

diff --git a/src/SonsOfPHP/Bard/src/Console/Command/SplitCommand.php b/src/SonsOfPHP/Bard/src/Console/Command/SplitCommand.php
@@ -17,11 +17,10 @@
  */
 final class SplitCommand extends AbstractCommand
 {
-    protected static $defaultName = 'split';
-
     protected function configure(): void
     {
         $this
+            ->setName('split')
             ->setDescription('Push changes to package repos using git subtree split')
             ->addOption('branch', null, InputOption::VALUE_REQUIRED, 'What branch we working with?', 'main')
             ->addOption('dry-run', null, InputOption::VALUE_NONE, 'Dry Run (Do not make any changes)')

diff --git a/src/SonsOfPHP/Bard/src/Console/Command/UpdateCommand.php b/src/SonsOfPHP/Bard/src/Console/Command/UpdateCommand.php
@@ -14,11 +14,10 @@
  */
 final class UpdateCommand extends AbstractCommand
 {
-    protected static $defaultName = 'update';
-
     protected function configure(): void
     {
         $this
+            ->setName('update')
             ->setDescription('Runs composer update for all packages')
         ;
     }