Skip to content
This repository has been archived by the owner on May 4, 2018. It is now read-only.

Read console fixes #877

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Read console fixes #877

wants to merge 10 commits into from

Conversation

jepickett
Copy link

Reworked pull request #873. Sending a return key to force ReadConsoleA to exit before closing the stdin handle. Fix only applies only for Win8+.

@Nodejs-Jenkins
Copy link

Thank you for contributing this pull request! Here are a few pointers to make sure your submission will be considered for inclusion.

Commit MSOpenTech/libuv@e2607ec2407797b4f87fbc052a7a3dda7ceb4411 has the following error(s):

  • First line of commit message must be no longer than 50 characters
  • Commit message must indicate the subsystem this commit changes

Commit MSOpenTech/libuv@363373dba61736e59211792584e8dda6687b7e0b has the following error(s):

  • First line of commit message must be no longer than 50 characters
  • Commit message must indicate the subsystem this commit changes

Commit MSOpenTech/libuv@ec6411a3996087e305d15d6fc77be5ccfe9ed914 has the following error(s):

  • Commit message must indicate the subsystem this commit changes

Commit MSOpenTech/libuv@8100f22a94e0252f9934925426a40bbe3aab094d has the following error(s):

  • Commit message must indicate the subsystem this commit changes

The following commiters were not found in the CLA:

  • Scott Blomquist

You can fix all these things without opening another issue.

Please see CONTRIBUTING.md for more information

@indutny
Copy link
Contributor

indutny commented Apr 4, 2014

Is it still relevant?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants