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

[Console][Process] Add $verbosity argument to mustRun helper method #20178

Closed
nicolas-grekas opened this issue Aug 30, 2024 · 3 comments
Closed

Comments

@nicolas-grekas
Copy link
Member

Q A
Feature PR symfony/symfony#58074
PR author(s) @willrowe
Merged in 7.2

We created this issue to not forget to document this new feature. We would really appreciate if you can help us with this task. If you are not sure how to do it, please ask us and we will help you.

To fix this issue, please create a PR against the 7.2 branch in the symfony-docs repository.

Thank you! 😃

@willrowe
Copy link

willrowe commented Sep 3, 2024

I'm not sure where in the docs this should be added. I can't find anywhere that mustRun is referenced in the process helper section.

@nicolas-grekas
Copy link
Member Author

Looks like we need doc for the method indeed!

@javiereguiluz
Copy link
Member

The mustRun() method is documented here: https://symfony.com/doc/current/components/process.html#usage

In any case, we don't document the verbose argument of run(), so I think we shouldn't document it here either. It's a nice feature, but it's easy to discover by users. Thanks.

@javiereguiluz javiereguiluz closed this as not planned Won't fix, can't repro, duplicate, stale Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants