-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Bump dependencies (so that users that install using Composer's --pref…
…er-lowest CLI option also get recent versions)
- Loading branch information
1 parent
2f06a16
commit 5c0a514
Showing
2 changed files
with
15 additions
and
15 deletions.
There are no files selected for viewing
This file contains 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
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
5c0a514
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.
thank you for the detailed commit message. I find the sentiment interessting.
was there a particular "error"/problem which motivated this change?
I would expect someone using "prefer-lowest" to get the oldest possible combination of dependencies which still work.
if the composer.json/lock before this commit did not work for some reason, I would expect phpunits CI to fail.
5c0a514
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.
"[T]he oldest possible combination of dependencies" still worked before this commit, but caused deprecation warnings with PHP 8.4-dev. This was brought up by @lolli42 in #5893.
5c0a514
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 see. does this mean it would be helpful to have a phpunit prefer lowest ci build which errors on deprecation warnings?
5c0a514
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 consider using testing with
--prefer-lowest
a waste of resources.