Remove engines
hueniverse opened this issue · comments
Eran Hammer commented
Jonathan Solis commented
Why remove 'engines' if the package isn't compatible with older versions of node? Tried pulling in this library on a node 6.11.0 project but it uses async/await, which isn't supported in that version. This would have at least provided that warning.
Colin Ihrig commented
It was removed because the engines
field doesn't help much. It only prints an easy to miss warning during npm install
.
Eran Hammer commented
Removed mostly due to inconsistent handling between npm and yarn.
lock commented
This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.