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

chore: update dependencies #23

Closed demurgos closed 5 years ago

demurgos commented 5 years ago

Why

The current dependencies have some vulnerabilities. It's also general good practice to keep the dependencies up to date. npm also added support for lock files, a feature enabling reproducible builds.

What

Comments

The new versions of the dependencies add a transitive dependency on Node 4+ (both cross-spawn and tap).

Other

demurgos commented 5 years ago

CI is failing on old Node versions, it is fixed in https://github.com/tapjs/foreground-child/pull/24.

bcoe commented 5 years ago

@demurgos mind rebasing your various pulls, and we can see if tests pass?

demurgos commented 5 years ago

@bcoe Done