chore(deps): update sabre/event PHP dependency [full-ci]#41676
Draft
phil-davis wants to merge 2 commits into
Draft
chore(deps): update sabre/event PHP dependency [full-ci]#41676phil-davis wants to merge 2 commits into
phil-davis wants to merge 2 commits into
Conversation
phil-davis
commented
Jul 11, 2026
phil-davis
commented
Contributor
Contributor
Author
|
There seems to be some problem with Firstly, apply owncloud/testing#225 to remove some noise from the log files. And I will run some failing scenarios locally to see what might be happening. |
Contributor
Author
|
Issue sabre-io/event#171 We do not need this patch release. The current I will look into this issue in my open-source contribution to the |
Loading composer repositories with package information Updating dependencies Lock file operations: 0 installs, 1 update, 0 removals - Upgrading sabre/event (5.1.8 => 5.1.9) Writing lock file
phil-davis
force-pushed
the
sabre-update-20260711
branch
from
47c7a64 to
83a07fa
Compare
Contributor
Author
|
Rebased and added a commit to set the priority of That has fixed CI. And added a specific changelog for the real code change. |
These two events had the same event priority. They happened to get in the right order because their event entries were sorted alphabetically, so "c" came before "h". Sabre\Event is no longer going to do this alphabetical sorting. It will trigger events in priority order, and then for events with the same priority, it will trigger the events in the order that they have been put into the listeners array. This will make the order of event execution more transparent. Signed-off-by: Phillip Davis <phil@jankaritech.com>
phil-davis
force-pushed
the
sabre-update-20260711
branch
from
83a07fa to
11ccfc1
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.