Skip to content

[PyMySQL] Fix stubs for pymysql.connections.Connection.__init__ - #14724

Merged
brianschubert merged 5 commits into
python:mainfrom
JanEricNitschke:address-stubtest-errors
Sep 15, 2025
Merged

brianschubert merged 5 commits into
python:mainfrom
JanEricNitschke:address-stubtest-errors

Conversation

@JanEricNitschke

Copy link
Copy Markdown
Contributor

@github-actions

This comment has been minimized.

@JanEricNitschke

Copy link
Copy Markdown
Contributor Author

Linking #14718 for visibility there

@github-actions

This comment has been minimized.

@brianschubert brianschubert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks! Unfortunately this has a behavior change (see below), but there's an easy fix

Comment thread stubs/PyMySQL/pymysql/connections.pyi Outdated
@github-actions

This comment has been minimized.

Comment thread stubs/PyMySQL/pymysql/connections.pyi Outdated
Comment thread stubs/PyMySQL/pymysql/connections.pyi Outdated
Comment thread stubs/PyMySQL/pymysql/connections.pyi Outdated
@github-actions

Copy link
Copy Markdown
Contributor

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

@brianschubert brianschubert left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nice, thank you!

@brianschubert
brianschubert merged commit 2081b9d into python:main Sep 15, 2025
48 checks passed
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.

2 participants