Skip to content

bpo-32658: Add Metacharacter (except \) are not active inside classes Regexpr HOWTO (GH-32230) #32230

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 2 commits into from
Apr 5, 2022

Conversation

MaxwellDupre
Copy link
Contributor

@MaxwellDupre MaxwellDupre commented Apr 1, 2022

Metacharacters (except \) are not active inside classes.
Bug Issue32658

Simple doc change so assume doesn't need NEWS addition.

https://bugs.python.org/issue32658

@bedevere-bot bedevere-bot added the docs Documentation in the Doc dir label Apr 1, 2022
@MaxwellDupre MaxwellDupre changed the title bpo-32658: Add Metacharacter (except \) are not active inside classes Regexpr HOWTO (GH-NNNN) bpo-32658: Add Metacharacter (except \) are not active inside classes Regexpr HOWTO (GH-32230) Apr 1, 2022
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.

Makes sense, \ is explicitly listed as a metacharacter in this doc.

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

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

@miss-islington
Copy link
Contributor

Sorry @MaxwellDupre and @JelleZijlstra, I had trouble checking out the 3.10 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 43571a3eea8b5931769376daf4bdad1c9184ae0d 3.10

@bedevere-bot
Copy link

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

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Apr 5, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Apr 5, 2022
Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
(cherry picked from commit 43571a3)

Co-authored-by: Mike cm <mikecmcleod@gmail.com>
@JelleZijlstra JelleZijlstra added needs backport to 3.10 only security fixes and removed needs backport to 3.10 only security fixes labels Apr 5, 2022
@miss-islington
Copy link
Contributor

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

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Apr 5, 2022
@bedevere-bot
Copy link

GH-32323 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 Apr 5, 2022
Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
(cherry picked from commit 43571a3)

Co-authored-by: Mike cm <mikecmcleod@gmail.com>
miss-islington added a commit that referenced this pull request Apr 5, 2022
Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
(cherry picked from commit 43571a3)

Co-authored-by: Mike cm <mikecmcleod@gmail.com>
miss-islington added a commit that referenced this pull request Apr 5, 2022
Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
(cherry picked from commit 43571a3)

Co-authored-by: Mike cm <mikecmcleod@gmail.com>
hello-adam pushed a commit to hello-adam/cpython that referenced this pull request Jun 2, 2022
Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
(cherry picked from commit 43571a3)

Co-authored-by: Mike cm <mikecmcleod@gmail.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