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

process: Updated example #3748

Merged
merged 6 commits into from
May 5, 2021
Merged

process: Updated example #3748

merged 6 commits into from
May 5, 2021

Conversation

MGlolenstine
Copy link
Contributor

If we don't supply .stdin(Stdio::piped()), the line 8 always panics on unwrap() on None.

Motivation

The example wasn't working when trying to copy-paste it over.

Solution

I updated the example, adding .stdin(Stdio::piped()) and import to fix the issue.

MGlolenstine and others added 2 commits May 4, 2021 15:35
If we don't supply `.stdin(Stdio::piped())`, the line `8` always panics on `unwrap() on None`.
@Darksonn Darksonn added A-tokio Area: The main tokio crate M-process Module: tokio/process T-docs Topic: documentation labels May 4, 2021
tokio/src/process/mod.rs Outdated Show resolved Hide resolved
tokio/src/process/mod.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@Darksonn Darksonn left a comment

Choose a reason for hiding this comment

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

Thanks.

@Darksonn Darksonn merged commit 541b0c3 into tokio-rs:master May 5, 2021
@MGlolenstine MGlolenstine deleted the patch-1 branch May 5, 2021 09:52
@Darksonn Darksonn mentioned this pull request May 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tokio Area: The main tokio crate M-process Module: tokio/process T-docs Topic: documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants