sveltejs / cli

The Svelte CLI
MIT License
261 stars 11 forks source link

fix: `tailwindcss` import insertions and execution order #221

Closed AdrianGonz97 closed 1 month ago

AdrianGonz97 commented 1 month ago

closes #220

This is a good instance where having runsAfter would've been useful. While tailwindcss doesn't require prettier to function, ensuring that it's executed after prettier would've guaranteed that the prettier-plugin-tailwindcss is installed when they've both been selected

rather than bringing runsAfter back, I've tweaked the sorting order so that 2 adders that don't have dependsOn specified stay in place, which is just a temp fix

Re: Duplicate @import statements - this was another case where " vs ' made a difference

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: be3a1cd31a0b8e3047d1e4678870b409e15bf087

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 1 month ago

Open in Stackblitz

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

commit: be3a1cd