nyc@14 update in devDependencies

[brodycj]

Platforms affected

All

Motivation and Context

This update resolves the npm audit warning at this time.

Description

see title: nyc@14 update in devDependencies

Testing

• npm run cover succeeds locally on Node.js versions 6, 8, and 10
• verified that Travis CI succeeds
• verified that AppVeyor CI succeeds - on Node.js versions 6 & 8, TBD still queued on Node.js 10

Checklist

• I've run the tests to see all new and existing tests pass
• I added automated test coverage as appropriate for this change
• Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
• If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
• I've updated the documentation if necessary

[raphinesse]

Fine by me if the failing AppVeyor CI run was just a hiccup.

[raphinesse]

I approved if the failing AppVeyor CI run was just a hiccup. I restarted the failing test on Node 10. But you did not wait for it to finish. Conditional approval seems to be a bad idea 😒

[brodycj]

Oh, I didn't realize you restarted it on Node.js 10. I should have asked for clarification, then.

I will watch the builds on the master branch, just in case.

[raphinesse]

I should have been clearer. I would not expect the update to break the tests, but it's definitely better to check the results on master. Thanks!

[brodycj]

This update still fails on a single end-to-end test on Node.js 10, on AppVeyor. I think we should revert this update and investigate why this failure persists. My bad.

[brodycj]

Reverting this PR, as I had proposed in PR #785, did not seem to resolve the test failure on Node.js 10 on AppVeyor. So I think this change should not be reverted, instead I will propose a quick test workaround.