fix: improve descriptions for --no-open and --no-open-target (#3151)

Author: snitin315

README.md

@@ -97,9 +97,11 @@ Options:
   --no-client-overlay        Do not show a full-screen overlay in the browser when there are compiler errors or warnings.
   --setup-exit-signals       Close and exit the process on SIGINT and SIGTERM.
   --no-setup-exit-signals    Do not close and exit the process on SIGNIT and SIGTERM.
-  --open [value]             Open the default browser, or optionally specify a browser name.
-  --open-app <value...>      Open specified browser.
-  --open-target <value...>   Open specified route in browser.
+  --open [value...]          Open the default browser.
+  --no-open                  Do not open the default browser.
+  --open-app <value>         Open specified browser.
+  --open-target [value...]   Open specified route in browser.
+  --no-open-target           Do not open specified route in browser.
   --client-logging <value>   Log level in the browser (none, error, warn, info, log, verbose).
   --history-api-fallback     Fallback to /index.html for Single Page Applications.
   --no-history-api-fallback  Do not fallback to /index.html for Single Page Applications.
@@ -108,6 +110,7 @@ Options:
   --public <value>           The public hostname/ip address of the server.
   --firewall [value...]      Enable firewall or set hosts that are allowed to access the dev server.
   --no-firewall              Disable firewall.
+  --watch-files <value...>   Watch static files for file changes.
 
 Global options:
   --color                    Enable colors on console.

bin/cli-flags.js

@@ -143,6 +143,7 @@ module.exports = {
         },
       ],
       description: 'Open the default browser.',
+      negatedDescription: 'Do not open the default browser.',
     },
     {
       name: 'open-app',
@@ -176,6 +177,7 @@ module.exports = {
         opts.open.target = opts.openTarget;
         delete opts.openTarget;
       },
+      negatedDescription: 'Do not open specified route in browser.',
       multiple: true,
     },
     {
@@ -254,7 +256,7 @@ module.exports = {
           type: 'string',
         },
       ],
-      description: 'Watch static files for file changes',
+      description: 'Watch static files for file changes.',
       multiple: true,
     },
   ],

test/cli/cli.test.js

@@ -246,6 +246,15 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--no-open', (done) => {
+    testBin('--no-open')
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
   it('--open-app google-chrome', (done) => {
     testBin('--open-app google-chrome')
       .then((output) => {
@@ -264,6 +273,15 @@ describe('CLI', () => {
       .catch(done);
   });
 
+  it('--no-open-target', (done) => {
+    testBin('--no-open-target')
+      .then((output) => {
+        expect(output.exitCode).toEqual(0);
+        done();
+      })
+      .catch(done);
+  });
+
   it('--open-target index.html', (done) => {
     testBin('--open-target index.html')
       .then((output) => {