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

update guzzlehttp/guzzle to 6.3.0 #8259

Merged
merged 6 commits into from
Feb 14, 2018
Merged

update guzzlehttp/guzzle to 6.3.0 #8259

merged 6 commits into from
Feb 14, 2018

Conversation

icewind1991
Copy link
Member

@icewind1991 icewind1991 commented Feb 8, 2018

@icewind1991 icewind1991 added the 2. developing Work in progress label Feb 8, 2018
@MorrisJobke
Copy link
Member

CI fails

@codecov
Copy link

codecov bot commented Feb 8, 2018

Codecov Report

Merging #8259 into master will decrease coverage by 0.06%.
The diff coverage is 89.28%.

@@             Coverage Diff              @@
##             master    #8259      +/-   ##
============================================
- Coverage     51.76%   51.69%   -0.07%     
- Complexity    25385    25391       +6     
============================================
  Files          1599     1599              
  Lines         95102    95119      +17     
  Branches       1376     1376              
============================================
- Hits          49227    49171      -56     
- Misses        45875    45948      +73
Impacted Files Coverage Δ Complexity Δ
lib/private/Files/Storage/DAV.php 65.18% <100%> (ø) 159 <0> (ø) ⬇️
lib/private/Http/Client/Response.php 92.3% <100%> (ø) 6 <1> (ø) ⬇️
lib/private/Http/Client/ClientService.php 100% <100%> (ø) 2 <0> (ø) ⬇️
lib/private/Http/Client/Client.php 95.08% <87.5%> (-4.92%) 21 <7> (+5)
lib/private/Files/ObjectStore/Swift.php 0% <0%> (-49.62%) 45% <0%> (ø)
...s/files_sharing/lib/Controller/ShareController.php 45.2% <0%> (-1.12%) 62% <0%> (+1%)
lib/private/Server.php 83.18% <0%> (-0.1%) 282% <0%> (ø)
lib/private/Security/CertificateManager.php 94% <0%> (+3%) 39% <0%> (ø) ⬇️

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 changed the title [wip] update guzzlehttp/guzzle to 6.3.0 update guzzlehttp/guzzle to 6.3.0 Feb 9, 2018
@icewind1991
Copy link
Member Author

ci looks happy, review time

Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

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

Seems to all still work as far as I can tell.

}

/**
* @param ResponseInterface $response
*/
private function extracRequestTokenFromResponse(ResponseInterface $response) {
private function extracRequestTokenFromResponse(\Psr\Http\Message\ResponseInterface $response) {
Copy link
Member

Choose a reason for hiding this comment

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

Either this is not needed, or the docs is now wrong

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed

Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 mentioned this pull request Feb 12, 2018
2 tasks
@icewind1991 icewind1991 added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Feb 13, 2018
@icewind1991 icewind1991 added this to the Nextcloud 14 milestone Feb 13, 2018
@oparoz
Copy link
Member

oparoz commented Feb 14, 2018

All ready to merge?

@icewind1991 icewind1991 mentioned this pull request Feb 14, 2018
2 tasks
@MorrisJobke MorrisJobke merged commit 24f9651 into master Feb 14, 2018
@MorrisJobke MorrisJobke deleted the guzzle6 branch February 14, 2018 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants