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
add _sqlite3 module #11174
base: main
Are you sure you want to change the base?
add _sqlite3 module #11174
Conversation
This aligns with the implementation while giving greater fidelity to runtime naming and inheritance Related to python#3968 and python#11141
This comment has been minimized.
This comment has been minimized.
|
According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉 |
| @@ -46,6 +46,7 @@ _pydecimal: 3.5- | |||
| _random: 2.7- | |||
| _sitebuiltins: 3.4- | |||
| _socket: 3.0- # present in 2.7 at runtime, but not in typeshed | |||
| _sqlite3: 2.7- | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| _sqlite3: 2.7- | |
| _sqlite3: 3.0- |
We just merged a PR removing all the 2.7 lines.
Also, general note: I was away for a few weeks and I see you have a number of open PRs that haven't been merged yet. I'm just glancing through my notifications at the moment, but when I find some time I'll try to go over open nondraft typeshed PRs to review what I can.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Appreciated, but no rush
This aligns with the implementation while giving greater fidelity to runtime naming and inheritance
Related to #3968 and #11141