My code will not run after importing ms about ms HOT 15 CLOSED

Ah yes, with 3.0 we will be dropping support for Node 12, since it's deprecated 👍

https://nodejs.org/en/about/releases/

from ms.

Does Repl.it require ESM code? I see an error about about CJS. If so, could you try the latest canary which ships support for ES Modules?

https://github.com/vercel/ms/releases/tag/3.0.0-canary.1

from ms.

Ah yes, with 3.0 we will be dropping support for Node 12, since it's deprecated 👍

Why?

since it is outdated

It's a terrible decision, since you may well support version 6

from ms.

having the same issue on Repl.it, did you manage to sort it out by any chance?

Update: Looks like I just had to upgrade the version of Node running on Repl.it as the default is v12

from ms.

I’m using [email protected] and run into the same issue in CJS code. The exported CJS code of ms is using optional chanining which is incompatible with Node.js 12. You must at least use Node.js v14 or later.

I guess v3 of this package uses Typescript and should be able to transpile the code being compatible with Node.js v12. I just don’t know enough about the build/transpilation process yet to make a suggestions how the current setup needs to be modified to work with Node.js v12.

from ms.

I digged through the scripts/build.js file searching the transpilation step causing this issue. It’s this part:

The snippet explicitly referencing ES2020 (Node.js v14).

Using ES2019 resolves to Node.js v12 and would map the engine requirement in the package.json file.

from ms.

But the non-canary release should still be working with Node 12 (latest stable).

from ms.

@leerob Ok thank you! Node 12 is in maintenance LTS until 2022-04-30. I hoped you would support it in 3.0 as well 😃

from ms.

Good point - we will chat about that before it becomes stable 😄

from ms.

Sweet! Thank you!

from ms.

Just to be clear @marcuspoehls, the only major change here is ESM and native types. So you shouldn't need to update, however I understand your point... and as it's probably an easy change, maybe we should add back in Node 12 support until LTS ends.

from ms.

@mrmckeb Hey Brody, the native types are the reason I wanted to update 😃 you have a nice typing setup (with template strings) making things just a bit nicer when using typescript.

I downgraded to [email protected] and created my own typings. I’ll probably upgrade to [email protected] next year when Node.js 12 is EOL. Assuming that you're targeting Node.js v14 with 3.x

from ms.

Use @lukeed/ms instead of this package.

from ms.

I agree with @mrmckeb, might make sense to wait until April to get rid of Node 12. Appreciate the feedback here!

from ms.

I believe we can close this out now. We are planning for the 3.0 stable release, which will include dropping Node 12 support since the LTS has passed. You can track that here: #184

from ms.