Cleanup

Signed-off-by: Sora Morimoto <sora@morimoto.io>

Author: smorimoto

dist/index.js

@@ -88,11 +88,9 @@ export async function installer() {
     core.exportVariable("DUNE_CACHE_TRANSPORT", "direct");
   }
   core.exportVariable("CLICOLOR_FORCE", "1");
-  const fnames = await retrieveOpamLocalPackages();
-  if (fnames.length > 0) {
-    if (OPAM_PIN) {
-      await pin(fnames);
-    }
+  if (OPAM_PIN) {
+    const fnames = await retrieveOpamLocalPackages();
+    await pin(fnames);
   }
   await exec("opam", ["--version"]);
   await exec("opam", ["exec", "--", "ocaml", "-version"]);

packages/setup-ocaml/src/installer.ts

@@ -137,6 +137,9 @@ export async function installOcaml(ocamlCompiler: string) {
 }
 
 export async function pin(fpaths: string[]) {
+  if (fpaths.length === 0) {
+    return;
+  }
   await core.group("Pin local packages", async () => {
     for (const fpath of fpaths) {
       const fname = path.basename(fpath, ".opam");