Add .nvmrc and .npmrc
#137
Conversation
.nvmrc
Outdated
| @@ -0,0 +1 @@ | |||
| v16.19.0 | |||
There was a problem hiding this comment.
Node.js 18 is the active line at the moment. See https://nodejs.dev/en/about/releases/.
We had lts/* in Gutenberg for quite some time, if I recall correctly. I'm not sure if there is something similar for engines in package.json. Anyway, LTS still meets the requirement >=16.0.0.
There was a problem hiding this comment.
I went with 16 because that's what's listed in package.json:
wordpress-playground/package.json
Lines 61 to 64 in 9c7d797
There was a problem hiding this comment.
Right, but it doesn't prevent using v18. It only sets the minimum version.
There was a problem hiding this comment.
Ok. I don't have a preference on which version is used.
There was a problem hiding this comment.
It's up to @adamziel then, it can be Node 16, too 😄
There was a problem hiding this comment.
I'd go with Node 18 just because it's an LTS release
There was a problem hiding this comment.
@adamziel Sounds good, updated.
|
Thank you! |
Fixes #135