node-core-utils icon indicating copy to clipboard operation
node-core-utils copied to clipboard

feat: add support for nvm

Open UlisesGascon opened this issue 2 years ago • 4 comments

UlisesGascon avatar Dec 04 '23 16:12 UlisesGascon

Not sure what's the point? We want the utils to work on all supported versions, so I think it's good if we have a variety of versions used by collaborators.

targos avatar Dec 04 '23 16:12 targos

Not sure what's the point? We want the utils to work on all supported versions, so I think it's good if we have a variety of versions used by collaborators.

I think that is important to use minimum compatible version when developing the library so we prevent to introduce bugs. For example using Node@20 features that are not available in 18.18.0.

The collaborators can use a variety of version defined by the engine https://github.com/nodejs/node-core-utils/blob/main/package.json#L7 independently 👍

UlisesGascon avatar Dec 20 '23 19:12 UlisesGascon

Ok, I won't block it but I don't use nvm so I'll let someone else review. Please if it lands, don't use feat: (or fix:) in the commit message, as this change doesn't impact users who install the package AFAIK.

targos avatar Dec 21 '23 07:12 targos

For example using Node@20 features that are not available in 18.18.0.

That should be checked by the CI instead, I believe. The config won't do much for contributors to this repo who don't use nvm, then they won't catch it anyway...

joyeecheung avatar Apr 26 '24 12:04 joyeecheung