-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Runtime] Update runtime.rst #19219
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
[Runtime] Update runtime.rst #19219
Conversation
typo PHP-PM:PHP-FPM
Good catch @morrsky! Thanks and congrats on your first Symfony Docs contribution! |
bcbbdcc
to
8962ac5
Compare
@@ -2,7 +2,7 @@ The Runtime Component | |||
===================== | |||
|
|||
The Runtime Component decouples the bootstrapping logic from any global state | |||
to make sure the application can run with runtimes like PHP-PM, ReactPHP, | |||
to make sure the application can run with runtimes like PHP-FPM, ReactPHP, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this change is wrong and PHP-PM refers to the PHP Process Manager.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Nyholm Can you help here as you contributed the initial version of this document?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Correct, It is meant to say PHP-PM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see #19221 where I try to clarify this by adding links to these projects
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry about this! I just merged your PR fixing this.
This PR was merged into the 5.4 branch. Discussion ---------- fix the Runtime component introduction see #19219 (comment) Commits ------- 817c172 fix the Runtime component introduction
typo PHP-PM:PHP-FPM