hapijs / code

Assertion library

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Remove engines

hueniverse opened this issue · comments

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.

It was removed because the engines field doesn't help much. It only prints an easy to miss warning during npm install.

Removed mostly due to inconsistent handling between npm and yarn.

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.