Skip to content

Commit

Permalink
fix: use info for venv/setuptools failures
Browse files Browse the repository at this point in the history
  • Loading branch information
aminya committed Nov 1, 2024
1 parent 255caeb commit 867ce44
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion dist/legacy/setup-cpp.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/legacy/setup-cpp.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/modern/setup-cpp.mjs

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/modern/setup-cpp.mjs.map

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions src/python/python.ts
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ async function setupVenv(foundPython: string) {
try {
await setupPipPackWithPython(foundPython, "venv", undefined, { upgrade: false, usePipx: false })
} catch (err) {
notice(`Failed to install venv: ${(err as Error).toString()}. Ignoring...`)
info(`Failed to install venv: ${(err as Error).toString()}. Ignoring...`)
}
}

Expand All @@ -85,7 +85,7 @@ async function setupWheel(foundPython: string) {
})
await setupPipPackWithPython(foundPython, "wheel", undefined, { upgrade: false, isLibrary: true, usePipx: false })
} catch (err) {
notice(`Failed to install setuptools/wheel: ${(err as Error).toString()}. Ignoring...`)
info(`Failed to install setuptools/wheel: ${(err as Error).toString()}. Ignoring...`)
}
}

Expand Down

0 comments on commit 867ce44

Please sign in to comment.