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

Updated python-engineio version for python-socketio compatibility #8649

Merged
merged 5 commits into from
May 28, 2021

Conversation

b-quachtran
Copy link
Contributor

@b-quachtran b-quachtran commented May 10, 2021

Proposed changes:

Status (please check what you already did):

  • added some tests for the functionality
  • updated the documentation
  • updated the changelog (please check changelog for instructions)
  • reformat files using black (please check Readme for instructions)

@b-quachtran b-quachtran requested a review from federicotdn May 12, 2021 18:20
@b-quachtran
Copy link
Contributor Author

Hey @federicotdn, I'm trying to update the engine-io dependency version but looks like a docs CI check is failing from the update. Is there anything that needs to be done other than running poetry update with the correct library version? Thanks!

changelog/8649.bugfix.md Outdated Show resolved Hide resolved
@federicotdn
Copy link
Contributor

@b-quachtran yes that should be enough! However the error looks related to us linking to a Sanic documentation section that no longer exists. Maybe this is already fixed on main?

@b-quachtran b-quachtran enabled auto-merge May 18, 2021 15:44
@tczekajlo tczekajlo disabled auto-merge May 28, 2021 06:43
@tczekajlo tczekajlo merged commit 9f682c0 into 2.3.x May 28, 2021
@tczekajlo tczekajlo deleted the socket-io-fix-2-3-X branch May 28, 2021 15:02
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