Skip to content

[Profiler] Improve a part of the profiler docs #20000

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

Merged

Conversation

lkolndeep
Copy link
Contributor

Improve the documentation with a link for the profile service declaration.

@carsonbot carsonbot added this to the 5.4 milestone Jun 27, 2024
@lkolndeep lkolndeep force-pushed the link_profile_service_declaration branch from b202c1a to 714a051 Compare June 27, 2024 21:22
@lkolndeep lkolndeep changed the title Improve the a part of the profiler docs Improve a part of the profiler docs Jun 27, 2024
@carsonbot carsonbot changed the title Improve a part of the profiler docs [Profiler] Improve a part of the profiler docs Aug 27, 2024
@javiereguiluz javiereguiluz merged commit 674f0c9 into symfony:5.4 Aug 27, 2024
3 checks passed
@javiereguiluz
Copy link
Member

Thanks @lkolndeep. I liked the idea of adding something here to explain where the profiler service comes from. However, while merging I reworded it to match the style of similar notes in other parts of the code. See 86a4de5

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