Skip to content

[Messenger] Clarify sequential execution of handlers using priority#21631

Merged
javiereguiluz merged 2 commits intosymfony:6.4from
sfinx13:patch-1
Feb 12, 2026
Merged

[Messenger] Clarify sequential execution of handlers using priority#21631
javiereguiluz merged 2 commits intosymfony:6.4from
sfinx13:patch-1

Conversation

@sfinx13
Copy link
Contributor

@sfinx13 sfinx13 commented Dec 8, 2025

This clarification improves developer understanding of handler priority behavior.
Thanks in advance for your review!

@sfinx13 sfinx13 changed the base branch from 8.0 to 7.3 December 8, 2025 20:57
@sfinx13 sfinx13 requested a review from xabbuh as a code owner December 8, 2025 20:57
@sfinx13 sfinx13 changed the base branch from 7.3 to 8.0 December 8, 2025 20:58
Copy link
Contributor

@ckrack ckrack left a comment

Choose a reason for hiding this comment

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

Added a suggestion to avoid a ; in the sentence.

I think this targets the wrong branch, handler priority was not introduced in 8.0

@javiereguiluz javiereguiluz changed the base branch from 8.0 to 6.4 February 12, 2026 12:14
@javiereguiluz javiereguiluz merged commit c1a17af into symfony:6.4 Feb 12, 2026
3 checks passed
@javiereguiluz
Copy link
Member

I like this proposal! It clarifies an important detail.

Thanks Saidi ... and sorry it took us so long to merge this.

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.

4 participants