From 77432327b09c6d0ab09483377008202c3e8bbcde Mon Sep 17 00:00:00 2001 From: Esteban Cairol Date: Mon, 8 Sep 2025 16:18:53 -0600 Subject: [PATCH] Reorder command arguments in Pressable_Site_WP_CLI_Command_Run class - Swapped the order of 'site' and 'wp-cli-command' arguments for clarity. - This order makes more sense and also matches a similar command for WPCOM wpcom:run-site-wp-cli-command - Also made Site a mandatory value, instead of optional --- commands/Pressable_Site_WP_CLI_Command_Run.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/commands/Pressable_Site_WP_CLI_Command_Run.php b/commands/Pressable_Site_WP_CLI_Command_Run.php index c0b1113..4569d11 100644 --- a/commands/Pressable_Site_WP_CLI_Command_Run.php +++ b/commands/Pressable_Site_WP_CLI_Command_Run.php @@ -61,8 +61,8 @@ protected function configure(): void { $this->setDescription( 'Runs a given WP-CLI command on a given Pressable site.' ) ->setHelp( 'This command allows you to run an arbitrary WP-CLI command on a Pressable site.' ); - $this->addArgument( 'wp-cli-command', InputArgument::REQUIRED, 'The WP-CLI command to run.' ) - ->addArgument( 'site', InputArgument::OPTIONAL, 'The domain or numeric Pressable ID of the site to open the shell to.' ); + $this->addArgument( 'site', InputArgument::REQUIRED, 'The domain or numeric Pressable ID of the site to open the shell to.' ) + ->addArgument( 'wp-cli-command', InputArgument::REQUIRED, 'The WP-CLI command to run.' ); $this->addOption( 'multiple', null, InputOption::VALUE_REQUIRED, 'Determines whether the `site` argument is optional or not. Accepted values are `all` or a comma-separated list of site IDs or domains.' ) ->addOption( 'skip-output', null, InputOption::VALUE_NONE, 'Skip outputting the response to the console.' );