Node version bump from 8 to 18 breaks Remix installation about web-streams-polyfill HOT 8 CLOSED

Sorry about the inconvenience!

IMHO a minimum node version bump should not be made in a minor release. We'll probably be able to work around this issue with overriding the dependency version but just wanted to let you know anyway! (I assume more remix users are going to run into this)

You're right about the engines bump, I shouldn't have done that. The polyfill should work fine when installed on Node 8, but building the polyfill requires a newer Node version (because of the build dependencies). I'll revert that.

Removing down-leveling is also a breaking change for TS users. Some users are stuck on older TS versions because of transpilation incompatibilities (see getsentry/sentry-javascript#2848 as an example we've run into). Is there appetite in making this a major version instead?

Hmm, I didn't think users would still be using such old TypeScript versions... I'll see what I can do.

from web-streams-polyfill.

Could you release a quick fix version for this? Maybe with the latest commit of 3.2.0? Some production systems are failing to build because of this.

from web-streams-polyfill.

Thanks for the quick turnaround!

from web-streams-polyfill.

Remove down-leveling for TypeScript 3.5 or lower

Removing down-leveling is also a breaking change for TS users. Some users are stuck on older TS versions because of transpilation incompatibilities (see getsentry/sentry-javascript#2848 as an example we've run into). Is there appetite in making this a major version instead?

from web-streams-polyfill.

whoops, sorry for the auto close, i just merged a workaround in our repo which accidentally closed this issue

from web-streams-polyfill.

Version 3.3.1 is out. Can you check if that fixes it?

from web-streams-polyfill.

Yes, it works now. Thank you.

from web-streams-polyfill.

Heads-up: I pushed a 3.3.2 release, since 3.3.1 was missing the dist/types/polyfill.d.ts file. 🤦‍♂️

from web-streams-polyfill.