Fix 2fa check (#128)

* Fix 2fa check * Add a ci check too
changesets · Jul 18, 2019 · 5b28c52 · 5b28c52
1 parent 6d11989
commit 5b28c52
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/.changeset/happy-llamas-speak/changes.json b/.changeset/happy-llamas-speak/changes.json
@@ -0,0 +1,4 @@
+{
+  "releases": [{ "name": "@changesets/cli", "type": "patch" }],
+  "dependents": []
+}
diff --git a/.changeset/happy-llamas-speak/changes.md b/.changeset/happy-llamas-speak/changes.md
@@ -0,0 +1 @@
+Fix 2FA check on release
diff --git a/packages/cli/src/commands/release/npm-utils.ts b/packages/cli/src/commands/release/npm-utils.ts
@@ -110,7 +110,7 @@ async function internalPublish(
   twoFactorState: TwoFactorState
 ): Promise<{ published: boolean }> {
   let publishFlags = opts.access ? ["--access", opts.access] : [];
-  if (twoFactorState.isRequired) {
+  if ((await twoFactorState.isRequired) && !isCI) {
     let otpCode = await getOtpCode(twoFactorState);
     publishFlags.push("--otp", otpCode);
   }