8000 csv, ctypes, configparser explanations by hatal175 · Pull Request #5204 · python/typeshed · GitHub
[go: up one dir, main page]

Skip to content

csv, ctypes, configparser explanations #5204

10000
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

Merged
merged 2 commits into from
Apr 11, 2021

Conversation

hatal175
Copy link
Contributor

explained/fixed stubtest exceptions for these.

Worth mentioning that Array.iter is there because of this issue:
python/mypy#2220

Copy link
Member
@JelleZijlstra JelleZijlstra left a comment

Choose a reason for hiding this comment

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

Thanks for all your work cleaning up the whitelists!

ctypes.memmove # CFunctionType
ctypes.memset # CFunctionType
ctypes.pointer # imported C function
ctypes.string_at # docstring argument name is wrong
Copy link
Member

Choose a reason for hiding this comment

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

Might want to report this as a CPython bug

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
JelleZijlstra added a commit that referenced this pull request Apr 11, 2021
I ran into this in #5204. This is already our de facto convention. I think it's confusing enough if not followed that it makes sense to standardize on this rule.

Should we add an example? It might be useful but it's not in line with the rest of the section.
@hatal175
Copy link
Contributor Author

Opened python bug:
https://bugs.python.org/issue43803

@JelleZijlstra JelleZijlstra merged commit 25bac1d into python:master Apr 11, 2021
@hatal175 hatal175 deleted the VariousStubExplanations branch April 11, 2021 03:20
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
0