search

IBM / nodejs-itoolkit

A JavaScript (Node.js) library for communicating with IBM i
MIT License
43 stars 37 forks source link

Upgrade minimum node engine? #354

Closed abmusse closed 1 year ago

abmusse commented 1 year ago

Currently we say minimum node 8. Which is way outdated now

https://github.com/IBM/nodejs-itoolkit/blob/265f4c54a05937d7f12fc7b1c1f157de23ff79b4/package.json#L62

markdirish commented 1 year ago

I don't think there's anything this package does that couldn't be node 8, but there are no guarantees for our dependencies. https://endoflife.date/nodejs says that Node.js 14, 16, and 18 are the current "support triad", so I guess 14 would make sense?

abmusse commented 1 year ago

I don't think there's anything this package does that couldn't be node 8

Facts. Nothing preventing Node 8 from being used

abmusse commented 1 year ago

No need to change this at the moment