Skip to content
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

[3.11] Add re.VERBOSE flag documentation example (GH-97678) #97882

Merged
merged 1 commit into from
Oct 5, 2022

Conversation

miss-islington
Copy link
Contributor

The current re.VERBOSE documentation example leaves space for ambiguous
interpretation. One may read that spaces within the (?: token are
spaces inside the non-capturing group (such as (?: )). This patch
removes the ambiguity by including examples after the statement.
(cherry picked from commit 0ceafa7)

Co-authored-by: Athos Ribeiro [email protected]

The current re.VERBOSE documentation example leaves space for ambiguous
interpretation. One may read that spaces within the `(?:` token are
spaces inside the non-capturing group (such as `(?: )`). This patch
removes the ambiguity by including examples after the statement.
(cherry picked from commit 0ceafa7)

Co-authored-by: Athos Ribeiro <[email protected]>
@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: 4 of 9 required status checks are expected..

1 similar comment
@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: 4 of 9 required status checks are expected..

@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: 2 of 9 required status checks are expected..

1 similar comment
@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: 2 of 9 required status checks are expected..

@miss-islington miss-islington merged commit 0151375 into python:3.11 Oct 5, 2022
@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Pull Request is not mergeable.

3 similar comments
@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Pull Request is not mergeable.

@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Pull Request is not mergeable.

@miss-islington
Copy link
Contributor Author

Sorry, I can't merge this PR. Reason: Pull Request is not mergeable.

@miss-islington miss-islington deleted the backport-0ceafa7-3.11 branch October 5, 2022 01:29
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 issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants