Skip to content

Conversation

@manuel3108
Copy link
Member

@manuel3108 manuel3108 commented Jan 12, 2025

Resolves #389

Don't use tinyexec for the git detection anymore, as it's doing strange stuff and completely bailing out of the program. This can be reproduced here: https://stackblitz.com/edit/node-jdza4knz?file=index.js. Below the code that is not working, you will also find an alternative that is directly invoking the default exec which works just fine.

This PR switches from tinyexec to default exec for the git detection

@changeset-bot
Copy link

changeset-bot bot commented Jan 12, 2025

🦋 Changeset detected

Latest commit: 2d410a3

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
Copy link

pkg-pr-new bot commented Jan 12, 2025

Open in Stackblitz

npm i https://pkg.pr.new/sveltejs/cli/sv@394
npm i https://pkg.pr.new/sveltejs/cli/svelte-migrate@394

commit: 2d410a3

@manuel3108 manuel3108 closed this Jan 12, 2025
@manuel3108 manuel3108 force-pushed the fix/addons-in-stackblitz branch from 077ab74 to 679ff83 Compare January 12, 2025 06:31
@manuel3108 manuel3108 reopened this Jan 12, 2025
@manuel3108 manuel3108 changed the title fix: add-ons in stackblitz fix: git detection inside preconditions failed Jan 12, 2025
@manuel3108 manuel3108 marked this pull request as ready for review January 12, 2025 06:45
@manuel3108 manuel3108 merged commit f1008b7 into main Jan 16, 2025
8 checks passed
@manuel3108 manuel3108 deleted the fix/addons-in-stackblitz branch January 16, 2025 17:34
@github-actions github-actions bot mentioned this pull request Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

npx sv add tailwindcss seems to do nothing

3 participants