Commands should accept stdin #80
Comments
|
With my plate, it will be while before this comes to the front of line for me. Whoever is able to take it I say go for it. |
nfischer
added a commit
that referenced
this issue
Oct 24, 2016
Read all of stdin and pass it to the underlying ShellJS command. This is also adds support for mocking stdin during tests, and refactors the mock utilities. Fixes #80
nfischer
added a commit
that referenced
this issue
Oct 30, 2016
Read all of stdin and pass it to the underlying ShellJS command. This is also adds support for mocking stdin during tests, and refactors the mock utilities. Fixes #80
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, it isn't possible to do something like
cat file.txt | shx grep 'foo'. It looks like shx isn't passing stdin along to the ShellJS command (although most commands actually support this).This is currently blocking my work on shelljs/shelljs#525 (I'd like to replace
.exec('grep...')with.exec('shx grep...')).This should be an easy fix, if someone wants to take it up. Looks like we'll only have to change these lines to call the shelljs function with a different
thisparameter (see the piping docs).The text was updated successfully, but these errors were encountered: