Skip to content

Conversation

@GreenTeaCake
Copy link

TLDR: Fixes #1953

Versions 1.6.0 and 1.6.1 of follow-redirects break Mocha test should support max redirects. The PR updates dependency version to the last compatible one.

@GreenTeaCake
Copy link
Author

@emilyemorehouse Build passes. Please consider merging as a temporary workaround to unblock CI verification.

@emilyemorehouse
Copy link
Member

Thank you!!

Any chance you've seen a changelog or something for follow-redirects? Did I miss it? I was hoping to find a list of possible breaking changes.

Copy link
Member

@emilyemorehouse emilyemorehouse left a comment

Choose a reason for hiding this comment

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

Celebratory taco 🌮🎉

@emilyemorehouse emilyemorehouse merged commit 2eeb59a into axios:master Feb 9, 2019
@GreenTeaCake GreenTeaCake deleted the fix-ci-build branch February 9, 2019 04:43
@GreenTeaCake
Copy link
Author

I'm sorry. I was unable to find the changelog either. :(

@realityking
Copy link

@emilyemorehouse These are the changes between 1.5.10 and 1.6.0: follow-redirects/follow-redirects@v1.5.10...v1.6.0

@axios axios locked and limited conversation to collaborators May 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

should support max redirects mocha test fails

5 participants