Skip to content

gh-88546: Make new paragraph for emphasis and reordered sentence #91614

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 3 commits into from
May 2, 2022

Conversation

slateny
Copy link
Contributor

@slateny slateny commented Apr 16, 2022

#88546

Also added a bit on Path.glob()

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Apr 16, 2022
@slateny slateny changed the title gh-88546: Made new paragraph for emphasis and reordered sentence gh-88546: Make new paragraph for emphasis and reordered sentence Apr 26, 2022
not by actually invoking a subshell.

Note that files beginning with a dot (``.``) can only be matched by :mod:`glob`
patterns that also start with a dot,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This could optionally be a bit more explicit, like

'... patterns that also start with a dot (so *. will not match), unlike ...'

Copy link
Member

Choose a reason for hiding this comment

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

I think this is clear enough as is.

Copy link
Member

@JelleZijlstra JelleZijlstra left a comment

Choose a reason for hiding this comment

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

Thanks, I have one small suggestion which I'll apply myself

not by actually invoking a subshell.

Note that files beginning with a dot (``.``) can only be matched by :mod:`glob`
patterns that also start with a dot,
Copy link
Member

Choose a reason for hiding this comment

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

I think this is clear enough as is.

@JelleZijlstra JelleZijlstra self-assigned this May 2, 2022
@JelleZijlstra JelleZijlstra added needs backport to 3.9 only security fixes needs backport to 3.10 only security fixes labels May 2, 2022
@JelleZijlstra JelleZijlstra merged commit b9ab6ce into python:main May 2, 2022
@miss-islington
Copy link
Contributor

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

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 2, 2022
…eordered sentence (pythonGH-91614)

(cherry picked from commit b9ab6ce)

Co-authored-by: slateny <46876382+slateny@users.noreply.github.com>
@bedevere-bot
Copy link

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

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label May 2, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 2, 2022
…eordered sentence (pythonGH-91614)

(cherry picked from commit b9ab6ce)

Co-authored-by: slateny <46876382+slateny@users.noreply.github.com>
@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label May 2, 2022
@bedevere-bot
Copy link

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

miss-islington added a commit that referenced this pull request May 2, 2022
…ed sentence (GH-91614)

(cherry picked from commit b9ab6ce)

Co-authored-by: slateny <46876382+slateny@users.noreply.github.com>
miss-islington added a commit that referenced this pull request May 2, 2022
…ed sentence (GH-91614)

(cherry picked from commit b9ab6ce)

Co-authored-by: slateny <46876382+slateny@users.noreply.github.com>
hello-adam pushed a commit to hello-adam/cpython that referenced this pull request Jun 2, 2022
…eordered sentence (pythonGH-91614)

(cherry picked from commit b9ab6ce)

Co-authored-by: slateny <46876382+slateny@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants