-
Notifications
You must be signed in to change notification settings - Fork 2
Specify the requirements #28
Comments
Indeed, I saw that the last major version of our generator doesn't have the engine specified in the package.json. I don't completely agree with the fact that we have to support a big range of Node's versions. Actually, the LTS (stable for prod) is in We have to readd the engine definition in the "engines": {
"node": ">=8.0.0",
"npm": ">=5.0.0"
}, |
+1 for defining the requirement in the package.json
I agree we must live in the present but think also of the past and don't drop to easily active/maintained version of nodejs without a good reason. |
The weird fact is that I still haven't Node 12 installed on my Mac (even on the last contrib) 😅 |
“it's broken” is not very relevant... What's broken exactly ? |
it looks like toolbox is not compatible with node 6 (use of trailing comma in function args list, use of async, etc).
It would be nice to see if we can be compatible with the maximum version of node and at least define with which version of nodejs this project is compatible.
The text was updated successfully, but these errors were encountered: