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

Added defaults.timeout to default config. #112

Merged
merged 1 commit into from Sep 24, 2015

Conversation

@bananaoomarang
Copy link
Contributor

commented Sep 24, 2015

This fixes #111, as suggested by @lovell.

Not sure what versions we want to support so haven't altered .travis.yml. Tests pass locally and fixes the issue for me.

mzabriskie added a commit that referenced this pull request Sep 24, 2015

Merge pull request #112 from bananaoomarang/timeout-fix
Added defaults.timeout to default config.

@mzabriskie mzabriskie merged commit 67a0fc3 into axios:master Sep 24, 2015

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 86.186%
Details
@krolow

This comment has been minimized.

Copy link

commented Sep 25, 2015

do you have this fix in any tag? i'm getting same error here

@mzabriskie

This comment has been minimized.

Copy link
Member

commented Sep 25, 2015

@krolow just barely merged it. Will be releasing a new version with fix soon.

@krolow

This comment has been minimized.

Copy link

commented Sep 25, 2015

alright looking for to get the release 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.