Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update types in node API: child_processes #2567

Closed
wants to merge 7 commits into from

Conversation

glennreyes
Copy link
Contributor

This PR updates all the types of node's child_process to the current supported node API's (v4+).

@glennreyes glennreyes force-pushed the node-child-processes branch from 3a78e21 to dbce552 Compare October 6, 2016 10:42
@glennreyes glennreyes force-pushed the node-child-processes branch from dbce552 to ea9c1f7 Compare October 9, 2016 19:24
@mroch mroch added the Library definitions Issues or pull requests about core library definitions label Oct 10, 2016
@gabelevi
Copy link
Contributor

Thanks for the contribution and sorry for the delay! I'll rerecord the tests and get this reviewed & merged ASAP!

@facebook-github-bot
Copy link
Contributor

@gabelevi has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@gabelevi
Copy link
Contributor

This is great, btw! Thanks a ton for doing this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Library definitions Issues or pull requests about core library definitions Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants