Skip to content

[WCM] [Components][Process] add documentation for the ProcessBuilder::enableUseExec() method #3999

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

[WCM] [Components][Process] add documentation for the ProcessBuilder::enableUseExec() method #3999

wants to merge 1 commit into from

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Jul 6, 2014

Q A
Doc fix? no
New docs? yes (symfony/symfony#11335)
Applies to 2.6+
Fixed tickets

@xabbuh xabbuh mentioned this pull request Jul 6, 2014
Closed
@xabbuh xabbuh changed the title [Components][Process] add documentation for the ProcessBuilder::enableUseExec() method [WCM] [Components][Process] add documentation for the ProcessBuilder::enableUseExec() method Jul 6, 2014
cordoval
cordoval reviewed Aug 24, 2014
View reviewed changes
components/process.rst

When you use the ``ProcessBuilder`` to build a ``Process``, you can use
the :method:`Symfony\\Component\\Process\\ProcessBuilder::enableUseExec`
method to prefix the executed script with "exec"::
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe something is missing here, the use case in which you would want this prefix done

Copy link
Member Author

Choose a reason for hiding this comment

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

The use case is listed in the issues that are linked above the new paragraph.

Copy link
Contributor

Choose a reason for hiding this comment

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

thanks, I have missed that one on the diff, thanks @xabbuh

@xabbuh xabbuh force-pushed the process-builder-use-exec branch from 75c3222 to 5dbc44e Compare September 4, 2014 06:41
@xabbuh
Copy link
Member Author

xabbuh commented Oct 3, 2014

Closing as the code pull request won't be merged.

@xabbuh xabbuh closed this Oct 3, 2014
@xabbuh xabbuh deleted the process-builder-use-exec branch October 3, 2014 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
On hold
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@xabbuh @cordoval @wouterj