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

Drop loop kwarg from asyncio.gather call #7

Merged
merged 1 commit into from
May 26, 2022

Conversation

mweinelt
Copy link
Contributor

On Python 3.10 gather does not accept the loop kwarg anymore:

>       res = await asyncio.gather(*requests, loop=loop, return_exceptions=True)
E       TypeError: gather() got an unexpected keyword argument 'loop'

Tested against python 3.9/3.10.

Copy link
Collaborator

@farmio farmio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi 👋!
Thank you very much!

We should probably remove the loop argument on line 275, and other occurrences, too.

@mweinelt
Copy link
Contributor Author

Indeed, updated.

@farmio
Copy link
Collaborator

farmio commented May 26, 2022

Does anyone have an idea why the 3.7 test runner fails here?

@mweinelt
Copy link
Contributor Author

Python 3.7.1

I'm willing to wage a guess that this is due to travis' outdated environment. Ideally the project would migrate to GitHub Actions or something else.

@mweinelt mweinelt mentioned this pull request May 26, 2022
@mweinelt
Copy link
Contributor Author

Tests fixed in #8.

On Python 3.10 gather does not accept the loop kwarg anymore:

>       res = await asyncio.gather(*requests, loop=loop, return_exceptions=True)
E       TypeError: gather() got an unexpected keyword argument 'loop'
@mweinelt
Copy link
Contributor Author

Rebased, so we benefit from the updated tests setup.

@farmio farmio merged commit 650622d into nielstron:master May 26, 2022
@mweinelt mweinelt deleted the py310-gather-drop-loop branch May 26, 2022 11:42
@iconnor iconnor mentioned this pull request Oct 6, 2023
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.

None yet

2 participants