feat(commonjs): Restore support for Node 10 after #1038

[niklasf]

Rollup Plugin Name: commonjs

This PR contains:

• bugfix
• feature
• refactor
• documentation
• other

Are tests included?

• yes (bugfixes and features will not be merged without tests)
• no, but feat(commonjs): support strict require semantics for circular imports #1038 added good test coverage

Breaking Changes?

• yes (breaking changes will not be merged unless absolutely necessary)
• no

Description

#1038 raised the required Node version of the commonjs plugin from 8 to 12, mentioning:

• minimum Node version raised to 12. This allows us to use "async await" (though 10 would have been enough for that...)

Since rollup itself is still usable with Node 10, it makes some sense to stick with 10 as long as reasonably convenient.

[shellscape]

Thanks for the PR, unfortunately our support policy is dictated by the Node LTS policy, which can be viewed here: https://nodejs.org/en/about/releases/

Rollup supporting v10+ is likely an oversight, since v10 dropped out of LTS a very long time ago.

[lukastaegert]

We are actively preparing Rollup v3 now, that raises the bar to Node 14 anyway

[niklasf]

Fair enough. Thanks for the quick response!