Supress rye install log output when in debug mode

rye uses [whatthestell](https://github.com/zekroTJA/whattheshell) [here](https://github.com/mitsuhiko/rye/blob/d950ca25b032d59846b78aba3dfb1284e3d04505/rye/src/cli/rye.rs#L392) to determine what shell is being used. This internally uses `ps` [here](https://github.com/zekroTJA/whattheshell/blob/b6290f96b5638314ac4547def37fe35985831d5d/src/infer/unix.rs#L52). My guess is that this prints`error: process ID out of range` to stderr when searching for the process which opened the shell in the current process "tree" because no shell is used when the `rye self install` command is executed by this typescript action.
eifinger · Feb 18, 2024 · d31db40 · d31db40
1 parent 43d2c88
commit d31db40
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
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/src/setup-rye.ts b/src/setup-rye.ts
@@ -174,6 +174,7 @@ async function installRye(
   core.debug(`Created temporary directory ${tempDir}`)
   const options: exec.ExecOptions = {
     cwd: tempDir,
+    silent: !core.isDebug(),
     env: {
       ...process.env,
       RYE_HOME: tempDir