Skip to content

Conversation

@miss-islington
Copy link
Contributor

@miss-islingtonmiss-islington commented Sep 26, 2022

Fix command line parsing: reject "-X int_max_str_digits" option with
no value (invalid) when the PYTHONINTMAXSTRDIGITS environment
variable is set to a valid limit.
(cherry picked from commit 4135166)

Co-authored-by: Victor Stinner vstinner@python.org

) Fix command line parsing: reject "-X int_max_str_digits" option with no value (invalid) when the PYTHONINTMAXSTRDIGITS environment variable is set to a valid limit. (cherry picked from commit 4135166) Co-authored-by: Victor Stinner <vstinner@python.org>
Copy link
Member

@vstinnervstinner left a comment

Choose a reason for hiding this comment

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

LGTM, good bot.

@miss-islington
Copy link
ContributorAuthor

@vstinner: Status check is done, and it's a failure or timed out ❌.

@ambv
Copy link
Contributor

ambv commented Oct 4, 2022

Test failing on Windows, looks legitimate: AssertionError: 0.015625 not less than 0.013671875. I'll re-run just in case.

@miss-islington
Copy link
ContributorAuthor

Status check is done, and it's a success ✅.

@miss-islington
Copy link
ContributorAuthor

Sorry, I can't merge this PR. Reason: You're not authorized to push to this branch. Visit https://docs.github.com/articles/about-protected-branches/ for more information..

@ambv
Copy link
Contributor

ambv commented Oct 4, 2022

...and it passed 🤔

@ambvambv merged commit 358b7a4 into python:3.9Oct 4, 2022
@miss-islingtonmiss-islington deleted the backport-4135166-3.9 branch October 4, 2022 18:57
Sign up for freeto 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

@miss-islington@ambv@vstinner@bedevere-bot