8000 bpo-46586: Fix documentation links by meersuri · Pull Request #31216 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

bpo-46586: Fix documentation links #31216

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.

Sign up for GitHub

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 4 commits into from
Feb 12, 2022
Merged

bpo-46586: Fix documentation links #31216

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a7dd04a
Fix enum.property documentation link
meersuri Feb 8, 2022
64f5b1d
📜🤖 Added by blurb_it.
blurb-it[bot] Feb 8, 2022
078b226
Fix links to builtin max, min in sys.float_info
meersuri Feb 10, 2022
47d95ac
Revert "📜🤖 Added by blurb_it."
meersuri Feb 12, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix links to builtin max, min in sys.float_info
  • Loading branch information
@meersuri
meersuri committed Feb 10, 2022
commit 078b226d28d39ddae12f448d95fe11c6db9165ea
4 changes: 2 additions & 2 deletions Doc/library/sys.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -545,15 +545,15 @@ always available.
| :const:`mant_dig` | DBL_MANT_DIG | float precision: the number of base-``radix`` |
| | | digits in the significand of a float |
+---------------------+----------------+--------------------------------------------------+
| :const:`max` | DBL_MAX | maximum representable positive finite float |
| :const:`!max` | DBL_MAX | maximum representable positive finite float |
Copy link
Member

Choose a reason for hiding this comment

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

I think this is better done in a separate PR, so we can backport to the other bugfix branches.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I see, so I guess that will require opening another issue and then the corresponding PR? Should I revert this commit here then?

Copy link
Member

Choose a reason for hiding this comment

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

Yes, revert the commit here. It doesn't need an issue; trivial docs fixes don't need an issue. You can just attach the second PR to the same issue.

Copy link
Member

Choose a reason for hiding this comment

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

Also, based on @zware's comment on the bug, it's going to be better to change these to just put max in double backticks, since they aren't supposed to link anywhere.

Copy link
Member

Choose a reason for hiding this comment

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

That comment was @merwok, not me 😉. I agree with him, though.

Copy link
Member

Choose a reason for hiding this comment

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

Oops, sorry!

+---------------------+----------------+--------------------------------------------------+
| :const:`max_exp` | DBL_MAX_EXP | maximum integer *e* such that ``radix**(e-1)`` is|
| | | a representable finite float |
+---------------------+----------------+--------------------------------------------------+
| :const:`max_10_exp` | DBL_MAX_10_EXP | maximum integer *e* such that ``10**e`` is in the|
| | | range of representable finite floats |
+---------------------+----------------+--------------------------------------------------+
| :const:`min` | DBL_MIN | minimum representable positive *normalized* float|
| :const:`!min` | DBL_MIN | minimum representable positive *normalized* float|
| | | |
| | | Use :func:`math.ulp(0.0) <math.ulp>` to get the |
| | | smallest positive *denormalized* representable |
Expand Down
0