-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
gh-107689: Add docstring to the ctypes array object #107697
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
gh-107689: Add docstring to the ctypes array object #107697
Conversation
Most changes to Python require a NEWS entry. Please add it using the blurb_it web app or the blurb command-line tool. |
Misc/NEWS.d/next/Documentation/2023-08-06-15-34-11.gh-issue-107689.oL193x.rst
Outdated
Show resolved
Hide resolved
- Add links to `__main__` and `sys.path` where appropriate - Ensure each paragraph never has more than one link to the same thing, to avoid visual clutter from too many links
…7689.oL193x.rst Co-authored-by: sunmy2019 <[email protected]>
@sunmy2019 done |
Co-authored-by: Charlie Zhao <[email protected]>
@CharlieZhao95 any updates on this? |
Thanks! Just a little reminder. The docstrings I've seen usually describe functions in the form of "what does this function can do", just like:
And it's rare to see "how can we use this function/type":
This is probably not a problem that needs to be modified, I know this docstring comes from the official documentation. Aside from this, LGTM! Next step you could wait for review from core devs :) |
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.
LG! We shall wait for core devs now.
Most changes to Python require a NEWS entry. Please add it using the blurb_it web app or the blurb command-line tool. |
Add a docstring to the ctypes array object. Docs from here were used.
ctypes.Array
#107689