Skip to content

Commit

Permalink
refactor: Improve variable naming
Browse files Browse the repository at this point in the history
  • Loading branch information
medikoo committed Jul 10, 2020
1 parent 585deb0 commit faccc70
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions lib/private/setup-dependency/install-external/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,9 @@ module.exports = async dependencyContext => {
latestSupportedPublishedVersion,
versionRange
} = dependencyContext;
const wasInstalled = await isDirectory(path);
const packageJson = wasInstalled ? getPackageJson(path) : null;
if (latestSupportedPublishedVersion && wasInstalled) {
const isInstalled = await isDirectory(path);
const packageJson = isInstalled ? getPackageJson(path) : null;
if (latestSupportedPublishedVersion && isInstalled) {
if (optionalChaining(packageJson, "version") === latestSupportedPublishedVersion) {
// Seems up to date, but let's follow with quick sanity check and confirm whether
// there are corresponding folders for subdependencies
Expand Down Expand Up @@ -65,7 +65,7 @@ module.exports = async dependencyContext => {
prepareDependency(name, targetVersion, externalContext)
);
if (!sourceDirname) return;
if (wasInstalled && packageJson && packageJson._npmCrossLinkCacheName) {
if (isInstalled && packageJson && packageJson._npmCrossLinkCacheName) {
const cachePackageJson = getPackageJson(sourceDirname);
if (
cachePackageJson &&
Expand All @@ -78,7 +78,7 @@ module.exports = async dependencyContext => {
log.notice("%s installing %s @ %s", dependentContext.name, name, targetVersion);

dependentContext.installationJobs.add(
`${ wasInstalled ? "update" : "install" }-dependency:${ name }`
`${ isInstalled ? "update" : "install" }-dependency:${ name }`
);
await copyDir(sourceDirname, path);

Expand Down

0 comments on commit faccc70

Please sign in to comment.