Fix: use async reading of stdin in bin/eslint.js

[baerrach]

closes #12212

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[ X] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:

See #12212

What changes did you make? (Give an overview)

Applied nodejs recommendations for processing stdin as per https://github.com/nodejs/node/blob/master/doc/api/process.md#processstdin

Is there anything you'd like reviewers to focus on?

Compatibility with other operating systems.
I've only tested this on Windows.

[mysticatea]

Thank you for your contribution.

Would you add a test for this?
CI has multiplatform tests, it's great if we run tests on there.

[baerrach]

I can, it'll take me a day or so to clear my work to find the time for this.

I'll have a look in the test cases, but all I need is a test that uses a large file (anything over 4k in size).

Does anyone know which test I can look at off the top of their heads?

[jsf-clabot]

All committers have signed the CLA.

[kaicataldo]

LGTM, thanks!

[mysticatea]

LGTM, thank you.

[mysticatea]

Ah, @jsf-clabot seems to report a problem. Would you check the e-mail address of your commits?

[baerrach]

@mysticatea that is because I pushed my changes from my work machine with a work email.

I then fixed that to my gmail account and force pushed to remove that work email.

Its now saying all checks passed, including licece/cla.

[mysticatea]

The bot comment has been fixed 5 hours ago.