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

Disable detaching sub-process on Windows #489

Merged

Conversation

stelcheck
Copy link
Contributor

Fixes #480

@coveralls
Copy link

coveralls commented Dec 15, 2017

Coverage Status

Coverage remained the same at 77.427% when pulling f8b2d1c on stelcheck:fixes/windows-detached-proc into 129153f on TypeStrong:master.

1 similar comment
@coveralls
Copy link

coveralls commented Dec 15, 2017

Coverage Status

Coverage remained the same at 77.427% when pulling f8b2d1c on stelcheck:fixes/windows-detached-proc into 129153f on TypeStrong:master.

@stelcheck
Copy link
Contributor Author

@blakeembrey this should solve the reported issue.

@dickon
Copy link

dickon commented Dec 19, 2017

I found ts-node with no arguments was exiting without giving me a repl on Windows and traced it down to the detached spawn not working as expected with stdio inherited. The source branch of this PR had the fix which came to mind (i.e. backs out for Windows to the behaviour before #419 came along), and so I switched to it and solved my problem, so looks good to me.

@ocboogie
Copy link

This fixed the issue for me 👍

stelcheck added a commit to stelcheck/ts-node that referenced this pull request Dec 20, 2017
This commit:

  1. Cleans up how the code is loaded
  2. Removes the necessity to use 'spawn'

Replaces TypeStrong#476
Replaces TypeStrong#489
@stelcheck stelcheck mentioned this pull request Dec 20, 2017
@blakeembrey blakeembrey merged commit 89d62aa into TypeStrong:master Dec 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants