Skip to content

Fix missing spaces with help for -m compileall -o, revisit #27591

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

Merged
merged 1 commit into from
Sep 17, 2021

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Aug 4, 2021

No description provided.

@blueyed blueyed changed the title Fix missing space with help for -m compileall -o Fix missing spaces with help for -m compileall -o, revisit Aug 4, 2021
@@ -367,9 +367,9 @@ def main():
'environment variable is set, and '
'"timestamp" otherwise.'))
parser.add_argument('-o', action='append', type=int, dest='opt_levels',
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe fix the dest= here too to be singular.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is actually a list.
I've fixed the fixup.

@ambv
Copy link
Contributor

ambv commented Aug 4, 2021

An issue and NEWS entry would be nice to have as well.

Copy link
Contributor Author

@blueyed blueyed left a comment

Choose a reason for hiding this comment

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

An issue and NEWS entry would be nice to have as well.

I do not think it is worth it, but can add it if required really, of course.

@@ -367,9 +367,9 @@ def main():
'environment variable is set, and '
'"timestamp" otherwise.'))
parser.add_argument('-o', action='append', type=int, dest='opt_levels',
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is actually a list.
I've fixed the fixup.

@serhiy-storchaka serhiy-storchaka added needs backport to 3.9 only security fixes needs backport to 3.10 only security fixes skip issue skip news type-bug An unexpected behavior, bug, or error labels Aug 25, 2021
@serhiy-storchaka
Copy link
Member

Documentation fixes usually do not need a NEWS entry.

@blueyed blueyed force-pushed the fix-compileall-help-space branch from 4408132 to 426245c Compare September 7, 2021 07:47
@ambv ambv merged commit 74cc245 into python:main Sep 17, 2021
@miss-islington
Copy link
Contributor

Thanks @blueyed for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

@bedevere-bot
Copy link

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 17, 2021
(cherry picked from commit 74cc245)

Co-authored-by: Daniel Hahler <git@thequod.de>
@bedevere-bot
Copy link

GH-28431 is a backport of this pull request to the 3.9 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Sep 17, 2021
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Sep 17, 2021
(cherry picked from commit 74cc245)

Co-authored-by: Daniel Hahler <git@thequod.de>
ambv pushed a commit that referenced this pull request Sep 17, 2021
(cherry picked from commit 74cc245)

Co-authored-by: Daniel Hahler <git@thequod.de>
ambv pushed a commit that referenced this pull request Sep 17, 2021
(cherry picked from commit 74cc245)

Co-authored-by: Daniel Hahler <git@thequod.de>
niyas-sait pushed a commit to niyas-sait/cpython that referenced this pull request Sep 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip issue skip news type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants