fix: properly log error when dependency install fails

sveltejs / cli

The Svelte CLI

MIT License

261 stars 11 forks source link

fix: properly log error when dependency install fails #228

Closed AdrianGonz97 closed 4 weeks ago

AdrianGonz97 commented 4 weeks ago

closes #227

changeset-bot[bot] commented 4 weeks ago

🦋 Changeset detected

Latest commit: d69760c5794cc1f40e9075b733803ff2bd9c0033

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

| Name | Type | | ---- | ----- | | sv | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

pkg-pr-new[bot] commented 4 weeks ago

Open in Stackblitz

pnpm add https://pkg.pr.new/sveltejs/cli/sv@228

commit: d69760c

AdrianGonz97 commented 4 weeks ago

I'm assuming this is technically a pnpm error for not printing to stderr, but we probably should care about both.

Yea, I think this is a pnpm bug. Tried testing out using the silent flag: pnpm i --silent and the process never ended. I think what we have now should be fine for the time being