Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix testsuite errors, due to a "Connection: upgrade" seen differently… #731

Merged
merged 1 commit into from
Jul 16, 2018

Conversation

LocutusOfBorg
Copy link
Contributor

… than "Connection: Upgrade"

@zaphoyd zaphoyd merged commit 5c48a11 into zaphoyd:master Jul 16, 2018
@zaphoyd
Copy link
Owner

zaphoyd commented Jul 16, 2018

Good catch, can't believe I missed this. This has been merged and included in the 0.8.1 release.

@LocutusOfBorg LocutusOfBorg deleted the testsuite-fix branch July 17, 2018 09:32
@LocutusOfBorg
Copy link
Contributor Author

Good catch, can't believe I missed this

I can't believe I did 2 Pull requests that have been accepted without request of changes :)...
I was really unsure about this one, I'm really happy I got the problem right (don't you have some travis-ci file to automatically run the testsuite on each commit?)

@zaphoyd
Copy link
Owner

zaphoyd commented Jul 17, 2018

Thats what I thought too.. but it seems to be broken. I’ll need to get that fixed.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants