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

[Stopwatch] Add ROOT constant to make it easier to reference #54854

Merged
merged 1 commit into from
May 21, 2024

Conversation

hacfi
Copy link
Contributor

@hacfi hacfi commented May 7, 2024

Q A
Branch? 7.1
Bug fix? no
New feature? no
Deprecations? no
Issues -
License MIT

Just an DX improvement so you can reference the root section with $stopwatch->getSectionEvents(Stopwatch::ROOT) instead of $stopwatch->getSectionEvents('__root__').

@carsonbot

This comment was marked as outdated.

@carsonbot
Copy link

It looks like you unchecked the "Allow edits from maintainer" box. That is fine, but please note that if you have multiple commits, you'll need to squash your commits into one before this can be merged. Or, you can check the "Allow edits from maintainers" box and the maintainer can squash for you.

Cheers!

Carsonbot

@OskarStark OskarStark changed the title [Stopwatch] Add ROOT constant to make it easier to reference [Stopwatch] Add ROOT constant to make it easier to reference May 8, 2024
Copy link
Member

@fabpot fabpot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For 7.2

@fabpot fabpot modified the milestones: 7.1, 7.2 May 11, 2024
@xabbuh
Copy link
Member

xabbuh commented May 13, 2024

What about if we also add a method named getRootSectionEvents()?

@hacfi
Copy link
Contributor Author

hacfi commented May 13, 2024

@xabbuh Updated as requested.

@fabpot
Copy link
Member

fabpot commented May 21, 2024

Thank you @hacfi.

@fabpot fabpot merged commit 8691af2 into symfony:7.2 May 21, 2024
7 of 10 checks passed
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.

None yet

4 participants