-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
gh-135676: Lexical analysis: Reword String literals and related sections #135942
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
Conversation
Co-authored-by: Blaise Pabon <blaise@gmail.com>
Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! This was linked into the PEP 750 documentation TODO list so I took a look. I have one comment as it relates to t-strings; the rest of the changes are broader in scope, so I'll leave it to others to comment. (FWIW, I like the reorganization here!)
@AA-Turner, ping, do you still want to review this? |
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
Thanks @encukou for the PR 🌮🎉.. I'm working now to backport this PR to: 3.14. |
Thanks @encukou for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13. |
… sections (pythonGH-135942) (cherry picked from commit 777159f) Co-authored-by: Petr Viktorin <encukou@gmail.com> Co-authored-by: Blaise Pabon <blaise@gmail.com> Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com> Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
Sorry, @encukou, I could not cleanly backport this to
|
GH-137048 is a backport of this pull request to the 3.14 branch. |
|
Hopefully this PR is not too big. I can split it up if that would make it easier to review.
The f-string section is untouched here; a stub is added for t-strings. I'll look at those next.
📚 Documentation preview 📚: https://cpython-previews--135942.org.readthedocs.build/