skip binary modules during cache reset

[kolontsov]

#100

[1999]

Good job! Would you mind writing a test for this case?

[kolontsov]

Thanks! No problem, but I have doubts about implementation: good test should use binary module, so we need to find it somewhere. Add something "small & binary" to devDependancies? Or compile own minimal 3-line module from source (it will require installing node-gyp npm)..

Any suggestion?

[1999]

Add something "small & binary" to devDependancies

Compiling our own module for this case seems to be a bit more fair, but unfortunately I'm not familiar with it. If it doesn't take a lot of time for you, it would be great. If it does, just add a small/existing binary module to devDependencies.

[kolontsov]

Here it is. npm install && npm test works locally, but travis build below failed without details (I guess they're not completely recovered from AWS outage yet). What I'm supposed to do? upd logs are available, checking.

[1999]

Yes, AWS can interfere in tests results. Let's just wait for tomorrow.

[kolontsov]

Test passes on all node.js versions except 'stable' 7.7.0, which was released today (2017-03-01).
They completely broke compiling addons: nodejs/node#11628. Waiting for a fix.

[kolontsov]

Node.js released fixed stable version, could you please restart Travis build?

[1999]

Nice one! Will release a new version today/tomorrow.