Skip to content

Update prepared process documentation #14006

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

Closed
wants to merge 1 commit into from

Conversation

mattstauffer
Copy link

The docs say that the prepared command line feature of the Process component uses a "double bracket" syntax; however, in looking at the code example and actually using it, @faxblaster think this should instead say "double quotes".

If your pull request documents a NEW FEATURE, use the same Symfony branch where the feature was introduced (and master for features of unreleased versions).

Apologies, I'm not actually sure in which version this was introduced; I'd be happy to re-pull it against a different branch if necessary. Thanks for understanding my ignorance!

The docs said it uses a "double bracket" syntax which I and @faxblaster think should be "double quotes".
@javiereguiluz
Copy link
Member

Matt, thanks a lot for this contribution! We merged the other related PR sent by @matthewtrask, but your changes were added to that PR too, so you are an important part of it too. Congrats on your first Symfony Docs contribution!

@mattstauffer mattstauffer deleted the patch-1 branch September 25, 2020 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants