uber / bufrw

Buffer Reading and Writing
MIT License
32 stars 19 forks source link

Do not upgrade npm in CI #66

Closed kriskowal closed 5 years ago

kriskowal commented 5 years ago

This change fixes the build. We have always upgraded npm as an initial step in CI. I believe this previously addressed a bug in the default version installed with node 0.10. However, now it brings in a version of npm that requires syntax introduced to JavaScript, oh, some years ago, which does not work in Node.js 0.10. We must continue testing in this version for…backward compatibility reasons.