Skip to content

Commit 502a50d

Browse files
committed
refactor: Use early return pattern
Signed-off-by: jongwooo <jongwooo.han@gmail.com>
1 parent 377c6da commit 502a50d

File tree

1 file changed

+18
-11
lines changed

1 file changed

+18
-11
lines changed

src/cache-utils.ts

Lines changed: 18 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -61,23 +61,30 @@ const getPackageManagerVersion = async (
6161
};
6262

6363
export const getPackageManagerInfo = async (packageManager: string) => {
64+
if (
65+
packageManager !== 'npm' &&
66+
packageManager !== 'pnpm' &&
67+
packageManager !== 'yarn'
68+
) {
69+
return null;
70+
}
71+
6472
if (packageManager === 'npm') {
6573
return supportedPackageManagers.npm;
66-
} else if (packageManager === 'pnpm') {
74+
}
75+
76+
if (packageManager === 'pnpm') {
6777
return supportedPackageManagers.pnpm;
68-
} else if (packageManager === 'yarn') {
69-
const yarnVersion = await getPackageManagerVersion('yarn', '--version');
78+
}
7079

71-
core.debug(`Consumed yarn version is ${yarnVersion}`);
80+
const yarnVersion = await getPackageManagerVersion('yarn', '--version');
81+
core.debug(`Consumed yarn version is ${yarnVersion}`);
7282

73-
if (yarnVersion.startsWith('1.')) {
74-
return supportedPackageManagers.yarn1;
75-
} else {
76-
return supportedPackageManagers.yarn2;
77-
}
78-
} else {
79-
return null;
83+
if (yarnVersion.startsWith('1.')) {
84+
return supportedPackageManagers.yarn1;
8085
}
86+
87+
return supportedPackageManagers.yarn2;
8188
};
8289

8390
export const getCacheDirectoryPath = async (

0 commit comments

Comments
 (0)