Skip to content

Add optional stream argument to render to#10

Open
joshwlewis wants to merge 1 commit into
SamVerschueren:masterfrom
joshwlewis:render-to-optional-stream
Open

Add optional stream argument to render to#10
joshwlewis wants to merge 1 commit into
SamVerschueren:masterfrom
joshwlewis:render-to-optional-stream

Conversation

@joshwlewis
Copy link
Copy Markdown

@joshwlewis joshwlewis commented Jun 8, 2020

As mentioned in SamVerschueren/listr-update-renderer#26, it makes sense to send human-readable and informational logging to stderr when designing a pipeable command. Those commands typically reserve stdout for data that could be sent to another command. This PR attempts to make this renderer useful in that scenario by adding an optional argument to send output to streams other than process.stdout, like process.stderr.

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.

1 participant