search

yeoman / update-notifier

Update notifications for your CLI app
BSD 2-Clause "Simplified" License
1.76k stars 132 forks source link

Update index.js #193

Closed erossignon closed 4 years ago

erossignon commented 4 years ago

Unfortunality version 5.0 has introduced a breaking change that prevent update-notifier working with nodejs version 8. although nodejs 8 is end of life, it is still widely use. This is easy to fix : replace catch { with catch(err) { to restore compatibility with nodejs 8.x.

sindresorhus commented 4 years ago

No thanks. While this is a simple fix, it's a slippery slope. Any sub-dependency can make a change that breaks Node.js 8 compatibility at any moment. I don't want to deal with that. The supported version is Node.js 10 and later. You can stay on version v4 if you need Node.js 8 compatibility.