Skip to content

really drop python<=3.7 support - #608

Open
kloczek wants to merge 2 commits into
MagicStack:masterfrom
kloczek:master
Open

really drop python<=3.7 support#608
kloczek wants to merge 2 commits into
MagicStack:masterfrom
kloczek:master

Conversation

@kloczek

Copy link
Copy Markdown

Filter all code over pyupgrade --py38-plus.

Filter all code over `pyupgrade --py38-plus`.
Signed-off-by: Tomasz Kłoczko <kloczek@github.com>
"requires-python = '>=3.8.0'" in pyproject.toml is enough.
Signed-off-by: Tomasz Kłoczko <kloczek@github.com>

@fantixfantix left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the PR! I feel neutral about most of the changes here, but still approving.

Comment threadsetup.py
Comment on lines -3 to -6
vi = sys.version_info
if vi < (3, 8):
raise RuntimeError('uvloop requires Python 3.8 or greater')

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

"requires-python = '>=3.8.0'" in pyproject.toml is enough.

I'd keep this extra check. For people who uses Python < 3.8, they may also have a pip around year 2016 or earlier, which doesn't honor requires-python.

Comment threadtests/test_pipes.py

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

unused import io now

@stankudrow

stankudrow commented Jan 9, 2025

Copy link
Copy Markdown

@kloczek , hello.

Any progress?

Sign up for freeto 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

@kloczek@stankudrow@fantix