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] Document mustRun() #4187

Closed
xabbuh opened this issue Aug 29, 2014 · 0 comments
Closed

[Process] Document mustRun() #4187

xabbuh opened this issue Aug 29, 2014 · 0 comments
Labels
actionable Clear and specific issues ready for anyone to take them. good first issue Ideal for your first contribution! (some Symfony experience may be required) Process
Milestone

Comments

@xabbuh
Copy link
Member

xabbuh commented Aug 29, 2014

Since symfony/symfony#8655, the Process component has a mustRun() method which can be used for convenience. It throws an exception when the command being run doesn't exit successfully (i.e. its exit code is 0).

@wouterj wouterj added this to the 2.5 milestone Aug 30, 2014
weaverryan added a commit that referenced this issue Oct 1, 2014
This PR was submitted for the master branch but it was merged into the 2.5 branch instead (closes #4201).

Discussion
----------

[Components][Process] `mustRun()` documentation

| Q             | A
| ------------- | ---
| Doc fix?      | no
| New docs?     | yes (symfony/symfony#8655)
| Applies to    | 2.5+
| Fixed tickets | #4187

@weaverryan Sorry, this should be merged into 2.5.

Commits
-------

ad6a340 [Components][Process] `mustRun()` documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
actionable Clear and specific issues ready for anyone to take them. good first issue Ideal for your first contribution! (some Symfony experience may be required) Process
Projects
None yet
Development

No branches or pull requests

3 participants