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] Clarify re docs for byte pattern group names (GH-99308) #101001

Merged
merged 1 commit into from
Mar 4, 2023

Conversation

Kentzo
Copy link
Contributor

@Kentzo Kentzo commented Jan 12, 2023

@slateny
Copy link
Contributor

slateny commented Jan 14, 2023

Per comment I don't believe there should be a backport for this.

@Kentzo
Copy link
Contributor Author

Kentzo commented Jan 14, 2023

#99308 (comment) says otherwise

Copy link
Member

@CAM-Gerlach CAM-Gerlach left a comment

Choose a reason for hiding this comment

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

Thanks @Kentzo . One set of suggestions, otherwise LGTM

Doc/library/re.rst Show resolved Hide resolved
Doc/library/re.rst Show resolved Hide resolved
Doc/library/re.rst Show resolved Hide resolved
Copy link
Contributor

@hauntsaninja hauntsaninja left a comment

Choose a reason for hiding this comment

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

Thanks for backporting this!

Looks like it's common enough for past versions of docs to contain "removed" for future versions of Python, so would you mind applying CAM's suggestions?

Edit: Since this has been lying around, I think this is fine to merge as is without the more specific directives; 3.12 isn't finalised and this seems like a short time between deprecation and removal.

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