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: python 3.6 compat in _message_generator #12

Merged
merged 1 commit into from
Aug 1, 2020

Conversation

pallas
Copy link
Contributor

@pallas pallas commented Aug 1, 2020

In e645297 feat: improve reconnection support (yet again), python3.6 got
broken because of a call to asyncio.create_task, which only exists in
python 3.7. The fix is to replace it with self._loop.create_task.

Signed-off-by: Derrick Lyndon Pallas derrick@meter.com

In `e645297 feat: improve reconnection support (yet again)`, python3.6 got
broken because of a call to `asyncio.create_task`, which only exists in
python 3.7.  The fix is to replace it with `self._loop.create_task`.

Signed-off-by: Derrick Lyndon Pallas <derrick@meter.com>
@pallas
Copy link
Contributor Author

pallas commented Aug 1, 2020

I swear I'm trying to move on to python 3.7!

@frederikaalund
Copy link
Collaborator

Thanks for creating this pull request.

In e645297 feat: improve reconnection support (yet again), python3.6 got
broken because of a call to asyncio.create_task, which only exists in
python 3.7

Ah, you're absolutely right. Sorry about that. I do try to maintain Python 3.6 compatibility but I must have missed that one.

Currently, there are no automated tests/builds and I only test on the latest stable Python version before release. An automated build/test system would definitely catch these issues but I don't have personal/commercial motivation to implement this. I'll be glad to accept any future pull requests that adds this.

I swear I'm trying to move on to python 3.7!

Backward compatibility is a burden but an important one. :) Thanks for making these pull requests.

@frederikaalund frederikaalund merged commit 3a67e5c into empicano:master Aug 1, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants