Skip to content
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

[3.12] gh-114014: Update fractions.Fraction()'s rational parsing regex (GH-114015) #114023

Merged
merged 2 commits into from
Jan 13, 2024

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Jan 13, 2024

Fix a bug in the regex used for parsing a string input to the fractions.Fraction constructor. That bug led to an inconsistent exception message being given for some inputs.


(cherry picked from commit dd56b57)

Co-authored-by: Crowthebird 78076854+thatbirdguythatuknownot@users.noreply.github.com
Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Mark Dickinson dickinsm@gmail.com

…ex (pythonGH-114015)

Fix a bug in the regex used for parsing a string input to the `fractions.Fraction` constructor. That bug led to an inconsistent exception message being given for some inputs.

---------

(cherry picked from commit dd56b57)

Co-authored-by: Crowthebird <78076854+thatbirdguythatuknownot@users.noreply.github.com>
Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Mark Dickinson <dickinsm@gmail.com>
@mdickinson
Copy link
Member

Looks like we need #114022 to be merged first to resolve the doc build issues.

@terryjreedy terryjreedy merged commit e081792 into python:3.12 Jan 13, 2024
24 checks passed
@terryjreedy
Copy link
Member

The doc fix was merged, CI green, so I merged this and 3.11.

@mdickinson
Copy link
Member

Thanks, @terryjreedy!

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