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

Force karma to use the latest 0.9.x working version of socket.io #1261

Closed
wants to merge 1 commit into from

Conversation

alias-mac
Copy link
Contributor

When running karma tests with ddescribe or iit, as well as in some other
cases, the browser gets disconnected for some reason.

The full information on this fix can be found in:
http://jamiolkowski.it/251/karma-runner-warn-chrome-35-0-1916-mac-os-x-10-8-5-disconnected-1-times/

When running karma tests with `ddescribe` or `iit`, as well as in some other
cases, the browser gets disconnected for some reason.

The full information on this fix can be found in:
http://jamiolkowski.it/251/karma-runner-warn-chrome-35-0-1916-mac-os-x-10-8-5-disconnected-1-times/
@alias-mac
Copy link
Contributor Author

We should probably start moving towards 1.x version of socket.io, but I'm not sure what is the roadmap on that.
Probably related with #1257 and #1258.

@vojtajina
Copy link
Contributor

Yes, there is a branch with upgrading to socket.io 1.0. In the meantime I think this is a good fix.

@alias-mac
Copy link
Contributor Author

Thanks @vojtajina, can you tell me what is the problem with the tests in travis ci? Seems that it passed the tests but then something else broke it, but I'm not sure what it is.
If you can shed me some light, I'll dig into it.

@maksimr
Copy link
Contributor

maksimr commented Dec 30, 2014

@alias-mac fixed problem with CI. It's not your fault

@vojtajina
Copy link
Contributor

Landed as 840ee5f

@vojtajina vojtajina closed this Dec 30, 2014
@vojtajina
Copy link
Contributor

Thank you @alias-mac

@alias-mac
Copy link
Contributor Author

Huge thanks!

@alias-mac alias-mac deleted the socket-io-version branch December 31, 2014 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants