python-hyper / wsproto

Sans-IO WebSocket protocol implementation
https://wsproto.readthedocs.io/
MIT License
261 stars 38 forks source link

Fix CI #184

Closed njsmith closed 1 year ago

njsmith commented 1 year ago

And hopefully will fix test failures like https://github.com/python-hyper/wsproto/actions/runs/4462959392/jobs/8252677125?pr=183

njsmith commented 1 year ago

......okay so this is a mess. As far as I can tell:

I think at this point we should give up on running the autobahn tests. They were really useful when we were bootstrapping to get confidence that we had things working at all, but at this point they're just breaking our CI and we've had plenty of times we've had to work around issues with them -- I don't really trust them for correctness any more than our code/tests.

njsmith commented 1 year ago

(we should also figure out if we want to do anything about our own wsaccel dependency, but that's a whole other issue)

njsmith commented 1 year ago

OK! I think this PR is finally ready!

@Kriechi I see you reviewed this just before I made the most-potentially-controversial change (removing autobahn), so I'll wait a bit to merge to give you (or anyone else) a chance to weigh in on that :-) (also b/c this currently isn't blocking anything AFAIK)