Allow publishing untagged releases

facebook · Jul 31, 2020 · 44a5106 · 44a5106
1 parent 3d08955
commit 44a5106
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 36 deletions.
diff --git a/scripts/release/publish-commands/confirm-version-and-tags.js b/scripts/release/publish-commands/confirm-version-and-tags.js
@@ -11,15 +11,19 @@ const theme = require('../theme');
 const run = async ({cwd, packages, tags}) => {
   clear();
 
-  if (tags.length === 1) {
+  if (tags.length === 0) {
     console.log(
-      theme`{spinnerSuccess ✓} You are about the publish the following packages under the tag {tag ${tags}}`
+      theme`{spinnerSuccess ✓} You are about the publish the following packages without any tags:`
+    );
+  } else if (tags.length === 1) {
+    console.log(
+      theme`{spinnerSuccess ✓} You are about the publish the following packages under the tag {tag ${tags}}:`
     );
   } else {
     console.log(
       theme`{spinnerSuccess ✓} You are about the publish the following packages under the tags {tag ${tags.join(
         ', '
-      )}}`
+      )}}:`
     );
   }
 

diff --git a/scripts/release/publish-commands/parse-params.js b/scripts/release/publish-commands/parse-params.js
@@ -30,39 +30,10 @@ const paramDefinitions = [
 
 module.exports = () => {
   const params = commandLineArgs(paramDefinitions);
-
-  const {skipPackages, tags} = params;
-
-  if (!tags || tags.length === 0) {
-    const usage = commandLineUsage([
-      {
-        content:
-          'Publishes the current contents of "build/node_modules" to NPM.',
-      },
-      {
-        header: 'Options',
-        optionList: paramDefinitions,
-      },
-      {
-        header: 'Examples',
-        content: [
-          {
-            desc: 'Dry run test:',
-            example: '$ scripts/release/publish.js --dry --tags next',
-          },
-          {
-            desc: 'Publish a new stable:',
-            example: '$ scripts/release/publish.js --tags next latest',
-          },
-        ],
-      },
-    ]);
-    console.log(usage);
-    process.exit(1);
+  if (!params.tags || !params.tags.length) {
+    params.tags = [];
   }
-
-  splitCommaParams(skipPackages);
-  splitCommaParams(tags);
-
+  splitCommaParams(params.skipPackages);
+  splitCommaParams(params.tags);
   return params;
 };