fixes #938, check install-modules (#941)

check `install-modules` even if `install-modules-with` is missing.
shogo82148 · Dec 12, 2021 · fc967a8 · fc967a8
1 parent b42ab90
commit fc967a8
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/src/cpan-installer.ts b/src/cpan-installer.ts
@@ -24,12 +24,8 @@ export interface Options {
 }
 
 export async function install(opt: Options): Promise<void> {
-  if (!opt.install_modules_with) {
-    core.info("nothing to install");
-    return;
-  }
   let installer: (opt: Options) => Promise<void>;
-  switch (opt.install_modules_with) {
+  switch (opt.install_modules_with || "cpanm") {
     case "cpanm":
       installer = installWithCpanm;
       break;
@@ -162,8 +158,10 @@ async function installWithCpanm(opt: Options): Promise<void> {
     args.push("--verbose");
   }
   args.push(...splitArgs(opt.install_modules_args));
-  if (await exists(path.join(workingDirectory, "cpanfile"))) {
-    await exec.exec(perl, [...args, "--installdeps", "."], execOpt);
+  if (opt.install_modules_with) {
+    if (await exists(path.join(workingDirectory, "cpanfile"))) {
+      await exec.exec(perl, [...args, "--installdeps", "."], execOpt);
+    }
   }
   const modules = splitModules(opt.install_modules);
   if (modules.length > 0) {