Update websockets library to work with Py3.9+#465
Merged
Conversation
Using websockets < 8.0 on Python >= 3.9 fails with a TypeError in the websockets library. However, version 8 of websockets drops support for older versions of Python that we still need to support, so this uses conditional dependencies to support both.
Member
|
Does this also help with #461 (not checked the differences supporting 3 9+)? |
Contributor
Author
|
@SimonRichardson Alas, no. The loop arguments will still need to be explicitly removed before 3.10 |
Member
|
!!build!! Try again... |
Member
|
!!build!! |
Member
|
!!build!! Sorry about the noise, just setting our jenkins CI python 3.9 ready. |
Member
|
@johnsca can you rebase your branch so we can land this. |
Member
|
!!build!! |
Member
|
@johnsca The reason the Not entirely sure this isn't being picked up for 3.9, but is for other versions. |
SimonRichardson
approved these changes
Feb 4, 2021
Member
|
|
Merged
jujubot
added a commit
that referenced
this pull request
Mar 31, 2021
#481 Merge 2.9 into master This is brings in 2.9-rc3 schema as well, so that we're dealing with the latest changes. Conflicts: _client3.py - just re-ran the `make client` as the code is autogenerated. 5e23162 (upstream/master) Merge pull request #480 from SimonRichardson/release-2.8.6 3e23b74 (origin/master, origin/HEAD, master) Merge pull request #479 from SimonRichardson/schema-update-2.8.10 7f96863 Merge pull request #478 from SimonRichardson/scale-typo fd1438c Merge pull request #475 from achilleasa/provide-accessor-for-machine-hostname-field 1015f4d Expand wait_for_idle to support waiting for status (#473) 7829fd0 Merge pull request #471 from SimonRichardson/release-2.8.5 fd36c8f Merge pull request #470 from juju/johnsca/bundle-charm-files-and-wait 3d91d6b Merge pull request #469 from juju/johnsca/accept-charm-files 4e4e108 Merge pull request #465 from juju/johnsca/update-websockets/py39 d696937 Merge pull request #467 from gnuoy/issue/466
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Using websockets < 8.0 on Python >= 3.9 fails with a TypeError in the websockets library. However, version 8 of websockets drops support for older versions of Python that we still need to support, so this uses conditional dependencies to support both.