review suggestions

zapier · Jan 8, 2025 · e649292 · e649292
1 parent 013147d
commit e649292
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 3 deletions.
diff --git a/packages/cli/docs/cli.md b/packages/cli/docs/cli.md
@@ -225,6 +225,7 @@ resources convert and relate to different actions.
 * `key-value pairs...` | The key-value pairs to set. Keys are case-insensitive. Each pair should be space separated and pairs should be separated by an `=`. For example: `A=123 B=456`
 
 **Flags**
+* `-f, --force` | Force the update of environment variables regardless if the app version is production or not. Use with caution.
 * `-d, --debug` | Show extra debugging output.
 
 **Examples**
@@ -242,6 +243,7 @@ resources convert and relate to different actions.
 * `keys...` | The keys to unset. Keys are case-insensitive.
 
 **Flags**
+* `-f, --force` | Force the update of environment variables regardless if the app version is production or not. Use with caution.
 * `-d, --debug` | Show extra debugging output.
 
 **Examples**

diff --git a/packages/cli/src/oclif/commands/env/set.js b/packages/cli/src/oclif/commands/env/set.js
@@ -100,7 +100,6 @@ SetEnvCommand.flags = buildFlags({
       char: 'f',
       description:
         'Force the update of environment variables regardless if the app version is production or not. Use with caution.',
-      hidden: true,
     }),
   },
 });

diff --git a/packages/cli/src/oclif/commands/env/unset.js b/packages/cli/src/oclif/commands/env/unset.js
@@ -61,6 +61,8 @@ class UnsetEnvCommand extends BaseCommand {
           `App version ${version} is the production version. Are you sure you want to unset potentially live environment variables?` +
             ` If so, run this command again with the --force flag.`,
         );
+      } else {
+        throw e;
       }
     }
 
@@ -84,7 +86,6 @@ UnsetEnvCommand.flags = buildFlags({
       char: 'f',
       description:
         'Force the update of environment variables regardless if the app version is production or not. Use with caution.',
-      hidden: true,
     }),
   },
 });

diff --git a/packages/cli/src/utils/api.js b/packages/cli/src/utils/api.js
@@ -97,7 +97,6 @@ const callAPI = async (
   }
 
   debug(`>> ${requestOptions.method} ${requestOptions.url || res.url}`);
-  debug(`>> Request Headers: ${JSON.stringify(requestOptions.headers)}`);
 
   if (requestOptions.body) {
     const replacementStr = 'raw zip removed in logs';