| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
There was a problem hiding this comment.
Nice! A couple of minor suggestions:
Sorry, something went wrong.
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
There was a problem hiding this comment.
A few more, sorry!
Sorry, something went wrong.
| * The :attr:`sys.version_info` value is now a named tuple, with attributes | ||
| named :attr:`!major`, :attr:`!minor`, :attr:`!micro`, | ||
| :attr:`!releaselevel`, and :attr:`!serial`. (Contributed by Ross | ||
| Light; :issue:`4285`.) | ||
|
|
||
| :func:`sys.getwindowsversion` also returns a named tuple, | ||
| with attributes named :attr:`major`, :attr:`minor`, :attr:`build`, | ||
| :attr:`platform`, :attr:`service_pack`, :attr:`service_pack_major`, | ||
| :attr:`service_pack_minor`, :attr:`suite_mask`, and | ||
| :attr:`product_type`. (Contributed by Brian Curtin; :issue:`7766`.) | ||
| with attributes named :attr:`!major`, :attr:`!minor`, :attr:`!build`, | ||
| :attr:`!platform`, :attr:`!service_pack`, :attr:`!service_pack_major`, | ||
| :attr:`!service_pack_minor`, :attr:`!suite_mask`, and | ||
| :attr:`!product_type`. (Contributed by Brian Curtin; :issue:`7766`.) |
There was a problem hiding this comment.
(It's possible to link to all of these attributes directly, I think -- but I agree that there's no need to do so; it would just be distracting for the reader. No change requested! :)
Sorry, something went wrong.
Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
There was a problem hiding this comment.
Thanks! Looks good other than my question at #115319 (comment)
Sorry, something went wrong.
There was a problem hiding this comment.
(Some line-length nits)
Sorry, something went wrong.
Always welcome, thank you for the review! |
Sorry, something went wrong.
There was a problem hiding this comment.
🚀
Sorry, something went wrong.
|
Thanks @hugovk for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12. |
Sorry, something went wrong.
|
Sorry, @hugovk, I could not cleanly backport this to 3.12 due to a conflict. cherry_picker 92483b21b30d451586c54dc4923665f7f7eedd7a 3.12 |
Sorry, something went wrong.
|
Sorry, @hugovk, I could not cleanly backport this to 3.11 due to a conflict. cherry_picker 92483b21b30d451586c54dc4923665f7f7eedd7a 3.11 |
Sorry, something went wrong.
|
GH-115330 is a backport of this pull request to the 3.12 branch. |
Sorry, something went wrong.
… related (pythonGH-115319) Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>. (cherry picked from commit 92483b2) Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
|
GH-115331 is a backport of this pull request to the 3.11 branch. |
Sorry, something went wrong.
… related (pythonGH-115319) Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>. (cherry picked from commit 92483b2) Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
python#115319) Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
| Back | FazBrowse Home | New Git URL |
Fixes most of the Sphinx reference warnings in whatsnew/2.7.rst, plus some related fixes in other files, 108 warnings in all:
The remaining 17 warnings in 2.7.rst might be things that need documenting; I've left them for another time:
📚 Documentation preview 📚: https://cpython-previews--115319.org.readthedocs.build/