Skip to content

Fix asyncio.start_server call (unexpected keyword argument 'loop')#2625

Merged
mkultraWasHere merged 2 commits into
masterfrom
clenk/py310-loop
Jul 28, 2022
Merged

Fix asyncio.start_server call (unexpected keyword argument 'loop')#2625
mkultraWasHere merged 2 commits into
masterfrom
clenk/py310-loop

Conversation

@clenk
Copy link
Copy Markdown
Contributor

@clenk clenk commented Jul 27, 2022

Description

The loop parameter was not necessary and was removed in Python 3.10. This was causing an error preventing the server from starting up at all in Python 3.10.

Fixes #2611.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Started up the server under both Python 3.10.5 and Python 3.7.5 and confirmed that the server started up without any related errors.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

The loop parameter was not necessary and was removed in Python 3.10.
@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@mkultraWasHere mkultraWasHere self-requested a review July 28, 2022 20:25
@mkultraWasHere mkultraWasHere merged commit 08e65a9 into master Jul 28, 2022
@mkultraWasHere mkultraWasHere deleted the clenk/py310-loop branch July 28, 2022 20:26
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.

unexpected keyword argument 'loop

2 participants