Skip to content

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.

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