Skip to content

Fixing Mocha tests.#1993

Merged
emilyemorehouse merged 1 commit intoaxios:masterfrom
GreenTeaCake:fix-ci-build
Feb 9, 2019
Merged

Fixing Mocha tests.#1993
emilyemorehouse merged 1 commit intoaxios:masterfrom
GreenTeaCake:fix-ci-build

Conversation

@GreenTeaCake
Copy link
Copy Markdown

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
Copy Markdown
Author

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

@emilyemorehouse
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Author

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

@realityking
Copy link
Copy Markdown

@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