| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
You could add this PEP to the release highlights: https://docs.python.org/3.12/whatsnew/3.12.html#summary-release-highlights |
Sorry, something went wrong.
There was a problem hiding this comment.
I'd like us to explicitly state somewhere that typing.TypeAliasType is not the type of typing.TypeAlias (and that in fact, they basically don't compose at all). I feel like this will inevitably cause confusion otherwise.
Sorry, something went wrong.
This allows :keyword:`type` to link to docs for the new `type` statement (being written in pythongh-104642) instead of to this header in the argparse docs.
There was a problem hiding this comment.
Other than #104642 (comment) and one tiny last nit, this looks fantastic. Thanks for the huge amount of effort here, and for putting up with my many rounds of review!
Sorry, something went wrong.
| of (str, str) -> str and (bytes, bytes) -> bytes. Also note\n\ | ||
| that if the arguments are instances of some subclass of str,\n\ | ||
| the return type is still plain str.\n\ | ||
| \n\ |
There was a problem hiding this comment.
Don't think two line breaks are necessary here
| \n\ |
Sorry, something went wrong.
There was a problem hiding this comment.
Thanks, fixed.
Also thanks for all the thorough reviews! I am going to merge this now so I can stop worrying about conflicts.
Sorry, something went wrong.
There was a problem hiding this comment.
Yup, it's time :)
Sorry, something went wrong.
|
Thanks @JelleZijlstra for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12. |
Sorry, something went wrong.
(cherry picked from commit 060277d) Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com> Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
|
GH-104989 is a backport of this pull request to the 3.12 branch. |
Sorry, something went wrong.
|
🎉 |
Sorry, something went wrong.
|
🥳🚀 |
Sorry, something went wrong.
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>. (cherry picked from commit 060277d) Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
|
GH-105007 is a backport of this pull request to the 3.11 branch. |
Sorry, something went wrong.
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | ||
|
|
||
| .. class:: FunctionDef(name, args, body, decorator_list, returns, type_comment) | ||
| .. class:: FunctionDef(name, type_params, args, body, decorator_list, returns, type_comment) |
There was a problem hiding this comment.
Just realised that type_params is in the wrong place here for the AST as it currently exists on main, where type_params is at the end. But probably not worth fixing that until questions about the ABI are resolved in #104974.
Sorry, something went wrong.
There was a problem hiding this comment.
Also, looks like you might have added type_params to the constructor for FunctionDef and ClassDef, but not AsyncFunctionDef!
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
📚 Documentation preview 📚: https://cpython-previews--104642.org.readthedocs.build/