-
Notifications
You must be signed in to change notification settings - Fork 168
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
Update CI test matrix #348
Update CI test matrix #348
Conversation
Thanks for the contribution! I don't think the failures are related but can you take a look and see if you can resolve them? |
I'm not quite sure how to address this issue properly. It seems that calling
will somehow cause the browser to actually visit https://example.com(:443). Thus trying to find the stub will fail. See d831973 for a provisional fix. |
@wynksaiddestroy I'm good with that fix, thanks! Looks like there are problems running on Ruby head though now. Can you take a look (or remove head)? |
It's because |
Looks like the other PR was merged but CI here is still failing. Please let me know when I should try again. |
There's no new version released yet. I just asked the author, if he plans to do it. |
@ronwsmith A new version of em-socksify just got released. Could you please try again? |
Huzzah, thanks for the contribution! |
As #342 seems to got stuck. Here's a new attempt: