Require Node 10+

[paulmelnikow]

BREAKING CHANGE

Close #1871

[paulmelnikow]

This won't turn green because Node 8 is still required on the repo. I think we probably can turn it off for the repo once this is merged, though if we want to leave it turned on, I guess we need to leave an empty Node 8 job in this branch.

[mastermatt]

If the idea is to only support LTS, this should be >=10.13.
https://nodejs.org/en/download/releases/

[paulmelnikow]

I think LTS and Active.

[mastermatt]

We should be supporting LTS Active and Maintenance.
https://nodejs.org/en/about/releases/
Otherwise we'd be dropping 10.x in two months.

[paulmelnikow]

I think the ones we support on that chart are Current, Active, and Maintenance. Does that sound right?

[paulmelnikow]

This won't turn green because Node 8 is still required on the repo. I think we probably can turn it off for the repo once this is merged, though if we want to leave it turned on, I guess we need to leave an empty Node 8 job in this branch.

Ah, I can turn it off for the beta branch.

[gr2m]

BREAKING CHANGE is not enough, you need to add a :. I know, tricky one, this happened to me before. But that's the reason why no new release was created for after this merge https://github.com/nock/nock/runs/437745060?check_suite_focus=true#step:5:65

[github-actions]

🎉 This PR is included in version 11.9.0-beta.1 🎉

The release is available on:

• npm package (@beta dist-tag)
• GitHub release

Your semantic-release bot 📦🚀