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

Revise passing state documentation #807

Closed
wants to merge 1 commit into from

Conversation

justinlevi
Copy link

Update the alternative approach to using returned results to be more inline with the suggestions proposed here:
#382

Using the Symfony Process component for returning simple system info.

Overview

This pull request:

Attempts to improve the documentation a little.

Update the alternative approach to using returned results to be more inline with the suggestions proposed here:
consolidation#382

Using the Symfony Process component for returning simple system info.
@greg-1-anderson
Copy link
Member

The focus of that section of the documentation is how to pass information from one task to another. The example of using PHP's exec is just a side point that one should only use defer for complex task collections. It's off-topic to diverge into a discussion on how to call the Symfony Process component here.

Optionally, if you wanted to put that example somewhere else, then we could include a link to it from this section.

If you do move this addition, I think you can use $process->run() instead of start() + wait(), and you are missing a $ on the last line of the code example.

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