Fix pnpm being detected as npm (#110)

* Fix pnpm being detected as npm

* Set version to 1.12.0-beta.2

* Tag npm packages as "next" immediately

Author: cknitt

.github/workflows/ci.yml

@@ -108,8 +108,8 @@ jobs:
         with:
           name: ${{ needs.build.outputs.artifact_filename }}
 
-      - name: Publish packages on npm with tag "ci"
+      - name: Publish packages on npm with tag "next"
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_ACCESS_TOKEN }}
         run: |
-          npm publish ${{ needs.build.outputs.artifact_filename }} --tag ci
+          npm publish ${{ needs.build.outputs.artifact_filename }} --tag next

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "create-rescript-app",
-  "version": "1.12.0-beta.1",
+  "version": "1.12.0-beta.2",
   "description": "Quickly create new ReScript apps from project templates.",
   "main": "out/create-rescript-app.cjs",
   "scripts": {

package.json

@@ -35,14 +35,14 @@ let getPackageManagerInfo = async () =>
     let filename = Path.parse(execPath).name->String.toLowerCase
 
     let packageManager = switch () {
-    | _ if filename->String.includes("npm") => Some(Npm)
     | _ if filename->String.includes("yarn") =>
       let versionResult = await Promisified.ChildProcess.exec(`${command} --version`)
       let version = versionResult.stdout->String.trim
       let isYarn1 = CompareVersions.compareVersions(version, "2.0.0")->Ordering.isLess
 
       Some(isYarn1 ? Yarn1 : YarnBerry)
     | _ if filename->String.includes("pnpm") => Some(Pnpm)
+    | _ if filename->String.includes("npm") => Some(Npm) // make sure this goes after pnpm ...
     | _ if filename->String.includes("bun") => Some(Bun)
     | _ => None
     }