Always set RYE_HOME to cached path (#188)

If rye was found in the tools cache of hosted runners RYE_HOME was not set. That led to the creation of the .rye folder in the user home.
eifinger · Feb 29, 2024 · aad0cd7 · aad0cd7
1 parent 6d859b6
commit aad0cd7
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 133 deletions.
diff --git a/dist/setup/index.js b/dist/setup/index.js
diff --git a/dist/setup/index.js.map b/dist/setup/index.js.map
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -33,8 +33,7 @@
     "@actions/glob": "^0.4.0",
     "@actions/io": "^1.1.3",
     "@actions/tool-cache": "^2.0.1",
-    "@octokit/rest": "^20.0.2",
-    "node-fetch": "^3.3.2"
+    "@octokit/rest": "^20.0.2"
   },
   "devDependencies": {
     "@types/node": "^20.11.20",

diff --git a/src/setup-rye.ts b/src/setup-rye.ts
@@ -242,13 +242,14 @@ async function installRye(
 
   const cachedPath = await tc.cacheDir(tempDir, 'rye', version, arch)
   core.info(`Moved Rye into ${cachedPath}`)
-  core.exportVariable('RYE_HOME', cachedPath)
   return cachedPath
 }
 
 function addRyeToPath(cachedPath: string): void {
   core.addPath(`${cachedPath}/shims`)
   core.info(`Added ${cachedPath}/shims to the path`)
+  core.exportVariable('RYE_HOME', `${cachedPath}/.rye`)
+  core.debug(`Set RYE_HOME to ${cachedPath}/.rye`)
 }
 
 function addMatchers(): void {