Skip to content

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
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.

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
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 <[email protected]>
@bedevere-bot
Copy link

GH-28431 is a backport of this pull request to the 3.9 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 <[email protected]>
ambv pushed a commit that referenced this pull request Sep 17, 2021
(cherry picked from commit 74cc245)

Co-authored-by: Daniel Hahler <[email protected]>
ambv pushed a commit that referenced this pull request Sep 17, 2021
(cherry picked from commit 74cc245)

Co-authored-by: Daniel Hahler <[email protected]>
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