fix: mark --open-app deprecated in favor of --open-app-name

webpack · Dec 10, 2021 · deb83bd · deb83bd
1 parent a23ea3f
commit deb83bd
Show file tree

Hide file tree

Showing 7 changed files with 13 additions and 7 deletions.
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/--open-app-name.",
         path: "open[].app",
       },
     ],
-    description: "Open specified browser.",
+    description:
+      "Open specified browser. Deprecated: please use open.app.name/--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/--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/--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/--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-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-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.