Skip to content

fix: Add test for SCP #2692

Merged
kylecarbs merged 3 commits into
mainfrom
scp
Jun 27, 2022
Merged

fix: Add test for SCP #2692
kylecarbs merged 3 commits into
mainfrom
scp

Conversation

@kylecarbs

Copy link
Copy Markdown
Member

This was hanging due to the stdin pipe never being closed.
A test has been added to make sure it works!

This was hanging due to the stdin pipe never being closed.
A test has been added to make sure it works!
@kylecarbs kylecarbs requested a review from johnstcn June 27, 2022 16:28
@kylecarbs kylecarbs self-assigned this Jun 27, 2022
@kylecarbs kylecarbs enabled auto-merge (squash) June 27, 2022 16:37

@johnstcn johnstcn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool library!

@kylecarbs kylecarbs merged commit d9da96c into main Jun 27, 2022
@kylecarbs kylecarbs deleted the scp branch June 27, 2022 16:41
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.

2 participants