-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Update icewind/streams and other libs #26762
Conversation
@PVince81, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975 to be a potential reviewer. |
stable9.1: owncloud-archive/3rdparty#284 |
Discussed with @DeepDiver1975 and he said he was ok with updating more libs. Now let's go and fix that t... OH NO, Jenkins is dead? https://jenkins.owncloud.org/job/owncloud-core/job/core/job/PR-26762/1/ |
Ok, what @DeepDiver1975 meant was actually owncloud-archive/3rdparty#284 (comment) for 9.1. Still, I think it should be fine to upgrade all libs on master at this stage. |
@DeepDiver1975 add your guzzle update here ? |
8188ec2
to
42acb3c
Compare
Rebased to reboot CI |
Tested, passed. Merge ? @DeepDiver1975 |
👍 |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
I wanted to upgrade icewind/streams only but composer decided to update more libs (minor versions), so why not.
The main goal is upgrade icewind/streams to get this PR icewind1991/Streams#5 which is a partial fix for #22370
@DeepDiver1975