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

gh-95605: Fix float(s) error message when s contains only whitespace #95665

Merged
merged 2 commits into from Aug 10, 2022

Conversation

mdickinson
Copy link
Member

@mdickinson mdickinson commented Aug 4, 2022

This PR fixes the error message from float(s) in the case where s contains only whitespace.

Closes #95605.

@mdickinson
Copy link
Member Author

mdickinson commented Aug 9, 2022

@ericvsmith Would you have bandwidth for review, by any chance? I think it's a fairly simple fix. (Indeed, I was sorely tempted to try a more complicated refactoring and cleanup, but I think that's best left for 3.12.)

Copy link
Member

@ericvsmith ericvsmith left a comment

This looks correct to me. Thanks, @mdickinson!

@mdickinson
Copy link
Member Author

mdickinson commented Aug 10, 2022

Thanks, @ericvsmith!

@mdickinson mdickinson merged commit 97e9cfa into python:main Aug 10, 2022
14 checks passed
@miss-islington
Copy link
Contributor

miss-islington commented Aug 10, 2022

Thanks @mdickinson for the PR 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒🤖 I'm not a witch! I'm not a witch!

@bedevere-bot
Copy link

bedevere-bot commented Aug 10, 2022

GH-95858 is a backport of this pull request to the 3.11 branch.

@bedevere-bot
Copy link

bedevere-bot commented Aug 10, 2022

GH-95859 is a backport of this pull request to the 3.10 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this issue Aug 10, 2022
…hitespace (pythonGH-95665)

This PR fixes the error message from float(s) in the case where s contains only whitespace.
(cherry picked from commit 97e9cfa75a80b54a0630b7371f35e368a12749d1)

Co-authored-by: Mark Dickinson <dickinsm@gmail.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Aug 10, 2022
…hitespace (pythonGH-95665)

This PR fixes the error message from float(s) in the case where s contains only whitespace.
(cherry picked from commit 97e9cfa)

Co-authored-by: Mark Dickinson <dickinsm@gmail.com>
mdickinson added a commit that referenced this issue Aug 10, 2022
…ace (GH-95665) (GH-95859)

This PR fixes the error message from float(s) in the case where s contains only whitespace.
(cherry picked from commit 97e9cfa)

Co-authored-by: Mark Dickinson <dickinsm@gmail.com>
mdickinson added a commit that referenced this issue Aug 11, 2022
…ace (GH-95665) (GH-95858)

This PR fixes the error message from float(s) in the case where s contains only whitespace.
(cherry picked from commit 97e9cfa)

Co-authored-by: Mark Dickinson <dickinsm@gmail.com>
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.

4 participants