fix: mark --open-app deprecated in favor of --open-app-name (#4091)

snitin315 · web-flow · commit 693c28a0499e · 2021-12-10T19:23:54.000+05:30
* fix: mark `--open-app` deprecated in favor of `--open-app-name`

* docs: update

* fix: update description
diff --git a/README.md b/README.md
@@ -160,7 +160,7 @@ Options:
   --no-open                                 Does not open the default browser.
   --open-target <value...>                  Opens specified page in browser.
   --open-app-name <value...>                Open specified browser.
-  --open-app <value...>                     Open specified browser.
+  --open-app <value...>                     Open specified browser. Deprecated: please use '--open-app-name'.
   --open-reset                              Clear all items provided in 'open' configuration. Allows to configure dev server to open the browser(s) and page(s) after server had been started (set it to true to open your default browser).
   --open-target-reset                       Clear all items provided in 'open.target' configuration. Opens specified page in browser.
   --open-app-name-reset                     Clear all items provided in 'open.app.name' configuration. Open specified browser.
diff --git a/bin/cli-flags.js b/bin/cli-flags.js
@@ -806,11 +806,13 @@ module.exports = {
       {
         type: "string",
         multiple: true,
-        description: "Open specified browser.",
+        description:
+          "Open specified browser. Deprecated: please use '--open-app-name'.",
         path: "open[].app",
       },
     ],
-    description: "Open specified browser.",
+    description:
+      "Open specified browser. Deprecated: please use '--open-app-name'.",
     simpleType: "string",
     multiple: true,
   },
diff --git a/lib/Server.js b/lib/Server.js
@@ -910,6 +910,7 @@ class Server {
 
     // https://github.com/webpack/webpack-dev-server/issues/1990
     const defaultOpenOptions = { wait: false };
+    // TODO: remove --open-app in favor of --open-app-name
     const getOpenItemsFromObject = ({ target, ...rest }) => {
       const normalizedOptions = { ...defaultOpenOptions, ...rest };
 
diff --git a/lib/options.json b/lib/options.json
@@ -580,7 +580,8 @@
             },
             {
               "type": "string",
-              "minLength": 1
+              "minLength": 1,
+              "description": "Open specified browser. Deprecated: please use '--open-app-name'."
             }
           ],
           "description": "Open specified browser."
diff --git a/test/__snapshots__/validate-options.test.js.snap.webpack4 b/test/__snapshots__/validate-options.test.js.snap.webpack4
@@ -520,7 +520,8 @@ exports[`options validate should throw an error on the "open" option with '{"app
       Details:
        * options.open.app should be an object:
          object { name?, arguments? }
-       * options.open.app should be a non-empty string."
+       * options.open.app should be a non-empty string.
+         -> Open specified browser. Deprecated: please use '--open-app-name'."
 `;
 
 exports[`options validate should throw an error on the "open" option with '{"foo":"bar"}' value 1`] = `
diff --git a/test/__snapshots__/validate-options.test.js.snap.webpack5 b/test/__snapshots__/validate-options.test.js.snap.webpack5
@@ -520,7 +520,8 @@ exports[`options validate should throw an error on the "open" option with '{"app
       Details:
        * options.open.app should be an object:
          object { name?, arguments? }
-       * options.open.app should be a non-empty string."
+       * options.open.app should be a non-empty string.
+         -> Open specified browser. Deprecated: please use '--open-app-name'."
 `;
 
 exports[`options validate should throw an error on the "open" option with '{"foo":"bar"}' value 1`] = `
diff --git a/test/cli/__snapshots__/basic.test.js.snap.webpack4 b/test/cli/__snapshots__/basic.test.js.snap.webpack4
@@ -116,7 +116,7 @@ Options:
   --no-open                                 Does not open the default browser.
   --open-target <value...>                  Opens specified page in browser.
   --open-app-name <value...>                Open specified browser.
-  --open-app <value...>                     Open specified browser.
+  --open-app <value...>                     Open specified browser. Deprecated: please use '--open-app-name'.
   --open-reset                              Clear all items provided in 'open' configuration. Allows to configure dev server to open the browser(s) and page(s) after server had been started (set it to true to open your default browser).
   --open-target-reset                       Clear all items provided in 'open.target' configuration. Opens specified page in browser.
   --open-app-name-reset                     Clear all items provided in 'open.app.name' configuration. Open specified browser.
diff --git a/test/cli/__snapshots__/basic.test.js.snap.webpack5 b/test/cli/__snapshots__/basic.test.js.snap.webpack5
@@ -114,7 +114,7 @@ Options:
   --no-open                                 Negative 'open' option.
   --open-target <value...>                  Opens specified page in browser.
   --open-app-name <value...>                Open specified browser.
-  --open-app <value...>                     Open specified browser.
+  --open-app <value...>                     Open specified browser. Deprecated: please use '--open-app-name'.
   --open-reset                              Clear all items provided in 'open' configuration. Allows to configure dev server to open the browser(s) and page(s) after server had been started (set it to true to open your default browser).
   --open-target-reset                       Clear all items provided in 'open.target' configuration. Opens specified page in browser.
   --open-app-name-reset                     Clear all items provided in 'open.app.name' configuration. Open specified browser.

Original file line number	Diff line number	Diff line change
`@@ -580,7 +580,8 @@`
`580`	`580`	`},`
`581`	`581`	`{`
`582`	`582`	`"type": "string",`
`583`		`- "minLength": 1`
	`583`	`+ "minLength": 1,`
	`584`	`+ "description": "Open specified browser. Deprecated: please use '--open-app-name'."`
`584`	`585`	`}`
`585`	`586`	`],`
`586`	`587`	`"description": "Open specified browser."`