-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
Update performance.rst #14489
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
Update performance.rst #14489
Conversation
Promoting the term "Stopwatch" to improve Google findability
@@ -211,8 +211,8 @@ deployment process too): | |||
|
|||
.. _profiling-applications: | |||
|
|||
Profiling Applications | |||
---------------------- | |||
Profiling Applications and Stopwatch |
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.
Shouldn't this be the following?
Profiling Applications and Stopwatch | |
Profiling Applications with Stopwatch |
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'm using the Stopwatch to measure how long a Console Command is taking to run. That's it. I wouldn't call that "profiling" - would you? So the message I'm trying to get across with the word "and" is: "It is a stopwatch; you can use it for profiling; but you can also just use it as a, well, stopwatch."
But I must admit that this message certainly doesn't get across fully ;-)
I have 3 counter arguments:
|
ping @javiereguiluz :-) |
See #14502. |
Promoting the term "Stopwatch" to improve Google findability, and also https://symfony.com/doc/4.4/index.html
See #14476