Merge pull request #820 from ocaml/fix-opam-selector

Fix selector to select the correct opam release asset

Author: smorimoto

CHANGELOG.md

@@ -8,6 +8,10 @@ and this project adheres to
 
 ## [unreleased]
 
+### Fixed
+
+- Fix asset selector to select the correct opam release asset.
+
 ## [3.0.0-beta]
 
 ### Changed

dist/index.js

@@ -42,9 +42,14 @@ export async function getLatestOpamRelease() {
       "Could not retrieve the opam release matching the version constraint",
     );
   }
-  const matchedAssets = latestRelease.assets.find((asset) =>
-    asset.browser_download_url.includes(`${ARCHITECTURE}-${PLATFORM}`),
-  );
+  const matchedAssets = latestRelease.assets.find((asset) => {
+    if (PLATFORM === "windows") {
+      return asset.browser_download_url.endsWith(
+        `${ARCHITECTURE}-${PLATFORM}.exe`,
+      );
+    }
+    return asset.browser_download_url.endsWith(`${ARCHITECTURE}-${PLATFORM}`);
+  });
   if (!matchedAssets) {
     throw new Error(
       "Could not find any assets matching the current platform or architecture",