Mention PCNTL extension installation #19026
Conversation
setup.rst
Outdated
| Symfony application locally. Also, you may install the `PCNTL`_ extension to | ||
| use some features like signal handling in | ||
| :doc:`the Messenger component </messenger>` and when using | ||
| :ref:`signals in commands <console_signal-event>`. |
There was a problem hiding this comment.
I don't think this is needed. There are more "optional extensions" in Symfony components. We should document these when you need it (like you did in this PR), but we shouldn't mention it in the general set-up guide (set-up should be quick and simple)
There was a problem hiding this comment.
I agree ... the setup.rst article is both special and very important. Every item we add to the "list of things to check" creates friction for new people using Symfony.
Given that this PCNTL extension is only needed in some very small and specific parts of some Symfony components, I wouldn't mention it in this article. Thanks.
There was a problem hiding this comment.
You're right, that's removed. Thank you for the review!
alexandre-daubois
force-pushed
the
pcntl-ext-doc
branch
from
4c82b26 to
88ad1cf
Compare
|
Very nice! Thanks a lot Alex. |
May fix #19006