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

Direct native piping #15861

Closed
wants to merge 7 commits into from
Closed

Direct native piping #15861

wants to merge 7 commits into from

Conversation

rjmholt
Copy link
Collaborator

@rjmholt rjmholt commented Aug 3, 2021

PR Summary

Resolves #1908.

Implements direct piping between native executables, preventing issues with stringification and encoding.

PR Context

PR Checklist

@ghost ghost assigned TravisEz13 Aug 3, 2021
@ghost ghost added the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Aug 7, 2021
@lygstate
Copy link

lygstate commented Aug 9, 2021

Good to see, that's what I want

@ghost ghost added Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept and removed Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept labels Aug 12, 2021
@daxian-dbw
Copy link
Member

A one-pager design doc would be very useful to review the design and code (if an RFC is too heavy in this case).

@ghost ghost added Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept and removed Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept labels Aug 17, 2021
@ghost ghost added the Stale label Sep 7, 2021
@ghost
Copy link

ghost commented Sep 7, 2021

This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 15 days. It will be closed if no further activity occurs within 10 days of this comment.

@ghost ghost closed this Sep 18, 2021
@lygstate
Copy link

anything to stop this to be implemented

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept
Projects
None yet
4 participants