-
-
Notifications
You must be signed in to change notification settings - Fork 31.1k
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-101100: Improve documentation on function attributes #112933
Conversation
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.
🧹📔📕📖📗📚📘📙📚📓📕🧹
And backport? |
Thanks @AlexWaygood for the PR, and @hugovk for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12. |
Sorry, @AlexWaygood and @hugovk, I could not cleanly backport this to
|
Sorry, @AlexWaygood and @hugovk, I could not cleanly backport this to
|
…ythonGH-112933)\n\nCo-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>.\n(cherry picked from commit 4c5b9c1)\n\n\nCo-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
…ythonGH-112933) (cherry picked from commit 4c5b9c1) Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com> Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>.
GH-112974 is a backport of this pull request to the 3.12 branch. |
@AlexWaygood There's a bit more to do to resolve for 3.11, can I leave this for you? |
Absolutely! (Though probably not tonight 😄) |
GH-113003 is a backport of this pull request to the 3.11 branch. |
…ython#112933) (cherry-picked from commit 4c5b9c1) Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
…#112933) Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
…#112933) Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
Theoretically, I could have kept the table as an old-style ASCII-art Sphinx table. But nearly every line was being changed anyway, so I figured I would update it to use the
.. list-table
syntax while I was here.This gets us down to 11 Sphinx warnings in the datamodel docs.
📚 Documentation preview 📚: https://cpython-previews--112933.org.readthedocs.build/