refactor: renamed the internal variable

openupm · Nov 16, 2020 · 78ede5b · 78ede5b
1 parent c033738
commit 78ede5b
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/lib/cmd-add.js b/lib/cmd-add.js
@@ -37,8 +37,8 @@ const add = async function(pkgs, options) {
 const _add = async function({ pkg, testables, force }) {
   // dirty flag
   let dirty = false;
-  // upstream flag
-  let useUpstream = false;
+  // is upstream package flag
+  let isUpstreamPackage = false;
   // parse name
   let { name, version } = parseName(pkg);
   // load manifest
@@ -60,7 +60,7 @@ const _add = async function({ pkg, testables, force }) {
     let pkgInfo = await fetchPackageInfo(name);
     if (!pkgInfo && env.upstream) {
       pkgInfo = await fetchPackageInfo(name, env.upstreamRegistry);
-      if (pkgInfo) useUpstream = true;
+      if (pkgInfo) isUpstreamPackage = true;
     }
     if (!pkgInfo) {
       log.error("404", `package not found: ${name}`);
@@ -126,7 +126,7 @@ const _add = async function({ pkg, testables, force }) {
       }
     }
     // pkgsInScope
-    if (!useUpstream) {
+    if (!isUpstreamPackage) {
       const [depsValid, depsInvalid] = await fetchPackageDependencies({
         name,
         version,
@@ -175,7 +175,7 @@ const _add = async function({ pkg, testables, force }) {
     // Log the existed package
     log.notice("manifest", `existed ${name}@${version}`);
   }
-  if (!useUpstream) {
+  if (!isUpstreamPackage) {
     // add to scopedRegistries
     if (!manifest.scopedRegistries) {
       manifest.scopedRegistries = [];