tapjs / foreground-child

Run a child as if it's the foreground process. Give it stdio. Exit when it exits.
http://tapjs.github.io/foreground-child/
ISC License
39 stars 14 forks source link

feat: require Node 6 #24

Closed demurgos closed 5 years ago

demurgos commented 5 years ago

Why

Supporting unmaintained Node versions is proving to be an increasing burden as third party libraries drop support for it (cross-spawn, tap). Using ES5 syntax while ES2015 brought a lot of improvements to the language is also a weight on maintenance. Finally, this is a pretty mild breaking change: there is no intent to break API compat. Since it implies a semver major update, it will be opt-in and only affect users explicitly updating their projects. The primary motivations for this PR is improving coverage tools using V8, a feature only present on modern Node versions anyway.

What

bcoe commented 5 years ago

@demurgos thank you for the first contribution.