Version 5 (major release) planning

sveltejs / svelte-preprocess

A ✨ magical ✨ Svelte preprocessor with sensible defaults and support for: PostCSS, SCSS, Less, Stylus, Coffeescript, TypeScript, Pug and much more.

MIT License

1.76k stars 151 forks source link

Version 5 (major release) planning #504

Closed dummdidumm closed 1 year ago

dummdidumm commented 2 years ago

This issue is intended to collect breaking changes we want to do for the next major version of svelte-preprocess. What currently comes to my mind:

remove support for the default language (deprecated already)
remove support for lang="typescript" and type="text/..." (deprecated already AFAIK)
remove support for Node 10 (EOL) and 12 (EOL end of April)
remove peer dependencies and instead tell in the docs what versions we tested it with. Npm 7 automatically installs peer dependencies, which means useless downloads for almost everyone. UPDATE: There's https://docs.npmjs.com/cli/v8/configuring-npm/package-json#peerdependenciesmeta so we can keep peer deps. We can do this now already.
Only support the new SASS API (right now we only support the legacy API). This means bumping the peer dep.

mcmxcdev commented 2 years ago

You might want to add another bullet point to consider switching from yarn v1, which has frozen development for a while, to e.g. pnpm (which would be in line with the package manager choice of other sveltejs repos). That's not a semver issue though.

kaisermann commented 2 years ago

This issue is intended to collect breaking changes we want to do for the next major version of svelte-preprocess. What currently comes to my mind:

I was hoping v5 would be released at a similar time as the RFCed new preprocess API, but it seems that's still far from happening. Removing deprecations and support for EOL node versions is a good start! We don't need to make it a very big breaking change 🤷

You might want to add another bullet point to consider switching from yarn v1, which has frozen development for a while, to e.g. pnpm (which would be in line with the package manager choice of other sveltejs repos).

I have no qualms with switching to another package manager, even more, if it's to the same one used by other Svelte projects.

frangio commented 2 years ago

Peer dependencies can be marked optional as shown here: https://docs.npmjs.com/cli/v8/configuring-npm/package-json#peerdependenciesmeta

I believe this turns off the automatic installs.