Skip to content

Commit f58488d

Browse files
committed
remove: the --debug flag from the share command
The flag is now defunct with the removal of the new CMD window. The flag only prevented the window from opening and forcing output to the terminal. This is NOT a breaking change.
1 parent 259de28 commit f58488d

3 files changed

Lines changed: 8 additions & 7 deletions

File tree

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
4545

4646
- Added new `CommandLine::shellExec` method to execute the command in the terminal, and also be able to return the output as a string to the calling method. This is just a wrapper around the native PHP `shell_exec` function.
4747

48+
### Removed
49+
50+
Removed the `--debug` flag from the `share` command since this is now defunct with the removal of the new CMD window. (Debug only prevented the CMD window opening, forcing errors to be logged directly to the terminal.) (Not a breaking change.)
51+
4852
## [3.1.6.1](https://github.com/yCodeTech/valet-windows/tree/v3.1.6.1) - 2025-03-11
4953

5054
### Fixed

cli/Valet/Ngrok.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,11 +51,10 @@ public function run(string $command) {
5151
/**
5252
* @param string $site The site
5353
* @param int $port The site's port
54-
* @param bool $debug Allow debug error output
5554
* @param array $options Options/flags to pass to ngrok
5655
* @return void
5756
*/
58-
public function start(string $site, int $port, $debug = false, array $options = []) {
57+
public function start(string $site, int $port, array $options = []) {
5958
if ($port === 443 && !$this->hasAuthToken()) {
6059
output('Forwarding to local port 443 or a local https:// URL is only available after you sign up.
6160
Sign up at: <fg=blue>https://ngrok.com/signup</>

cli/valet.php

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -797,10 +797,8 @@
797797
* - `-o [option]`
798798
* ---
799799
* `valet share mysite --options domain=example.com//region=eu//request-header-remove="blah blah"`
800-
*
801-
* @param bool $debug Allow error messages to output to the current terminal
802800
*/
803-
$app->command('share [site] [-o|--options=] [--debug]', function ($input, $site = null, $options = null, $debug = null) {
801+
$app->command('share [site] [-o|--options=]', function ($input, $site = null, $options = null) {
804802

805803
// Send an error if the option shortcut has a =
806804
if (str_contains($input, "-o=")) {
@@ -811,7 +809,7 @@
811809

812810
$options = $options != null ? explode("//", $options) : [];
813811

814-
Ngrok::start($url, Site::port($url), $debug, $options);
812+
Ngrok::start($url, Site::port($url), $options);
815813

816814
})->descriptions('Share the current working directory site with a publically accessible URL', [
817815
"site" => "Optionally, the site name",
@@ -1365,4 +1363,4 @@
13651363
/**
13661364
* Run the application.
13671365
*/
1368-
$app->run();
1366+
$app->run();

0 commit comments

Comments
 (0)