svelte-preprocess icon indicating copy to clipboard operation
svelte-preprocess copied to clipboard

Published 20 hours ago • verified publisher icon sveltejs

Version 5 (major release) planning

Open dummdidumm opened this issue 2 years ago • 3 comments

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.

dummdidumm avatar Apr 14 '22 10:04 dummdidumm

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.

mcmxcdev avatar May 06 '22 11:05 mcmxcdev

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.

kaisermann avatar May 06 '22 15:05 kaisermann

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.

frangio avatar Jun 24 '22 16:06 frangio