FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.13] gh-123968: Fix lower bound for `python -m random --float` (GH-123971) by miss-islington · Pull Request #124009 · python/cpython · GitHub

/ cpython Public

[3.13] gh-123968: Fix lower bound for python -m random --float (GH-123971) - #124009

Merged
Yhg1s merged 1 commit into
python:3.13from
miss-islington:backport-a362c41-3.13
Sep 23, 2024
Merged

[3.13] gh-123968: Fix lower bound for python -m random --float (GH-123971)#124009
Yhg1s merged 1 commit into
python:3.13from
miss-islington:backport-a362c41-3.13

Conversation

miss-islington commented Sep 12, 2024
edited by hugovk
Loading

Copy link
Copy Markdown
Contributor

Fixes #123968.

(cherry picked from commit a362c41)

Co-authored-by: Anders Kaseorg andersk@mit.edu

…onGH-123971)

(cherry picked from commit a362c41)

Co-authored-by: Anders Kaseorg <andersk@mit.edu>

Yhg1s commented Sep 23, 2024

Copy link
Copy Markdown
Member

Given that this is a new feature, yeah, let's get the fix into 3.13.0 so people don't end up relying on the less desirable behaviour.

Yhg1s merged commit 7b8affa into python:3.13 Sep 23, 2024

Copy link
Copy Markdown
Contributor

The current docs do not reflect this fix, yet:
https://docs.python.org/3.13/library/random.html#random-cli

-f
--float
Print a random floating-point number between 1 and N inclusive, using uniform().

hugovk commented Sep 25, 2024

Copy link
Copy Markdown
Member

It's because this PR did not update the docs.

@mahlzahn, would you like to create a PR to fix it?

This is the file to edit:
https://github.com/python/cpython/blob/main/Doc/library/random.rst

No need for a news entry, and we can reuse issue number gh-123968 for the PR title prefix.

nanjekyejoannah pushed a commit that referenced this pull request Sep 25, 2024
* fix -f/--float command line option description

See gh-123968 gh-124009

* Update Doc/library/random.rst

---------

Co-authored-by: Sergey B Kirpichev <skirpichev@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 25, 2024
…onGH-124517)

* fix -f/--float command line option description

See pythongh-123968 pythongh-124009

* Update Doc/library/random.rst

---------

(cherry picked from commit f8651a2)

Co-authored-by: Robert Wolff <mahlzahn@posteo.de>
Co-authored-by: Sergey B Kirpichev <skirpichev@gmail.com>
Yhg1s pushed a commit that referenced this pull request Sep 26, 2024
…124517) (#124523)

gh-123968: fix -f/--float command line option description (GH-124517)

* fix -f/--float command line option description

See gh-123968 gh-124009

* Update Doc/library/random.rst

---------

(cherry picked from commit f8651a2)

Co-authored-by: Robert Wolff <mahlzahn@posteo.de>
Co-authored-by: Sergey B Kirpichev <skirpichev@gmail.com>
miss-islington deleted the backport-a362c41-3.13 branch January 2, 2026 18:04
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL