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

Fixing Mocha tests. #1993

Merged
merged 1 commit into from Feb 9, 2019
Merged

Fixing Mocha tests. #1993

merged 1 commit into from Feb 9, 2019

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