-
-
Notifications
You must be signed in to change notification settings - Fork 31.4k
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
gh-118235: Skip RAISE_SYNTAX_ERROR rules in the grammar spec #118237
Conversation
Another alternative it's to move all these rules to be inside an |
Thanks for the suggestion, I'll look into that! I'm limiting the time I spend on grammar-in-docs; with beta1 and PyCon US coming up it might be a month before my next post here. |
In any case I think meanwhile this is not a bad temporary solution, so let's merge it for now but let's keep an eye on the better one 👍 |
Thanks @encukou for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12. |
…ythonGH-118237) (cherry picked from commit ef940de) Co-authored-by: Petr Viktorin <encukou@gmail.com>
GH-118309 is a backport of this pull request to the 3.12 branch. |
… spec (pythonGH-118237)" This reverts commit ef940de. The highlighter change is no longer needed, since all RAISE_SYNTAX_ERROR actions are in invalid rules.
I'm not too comfortable with regular expressions, and this looks like piling on another hack, but this highlighter rule strips all the current
RAISE_SYNTAX_ERROR
alternatives:A diff of the rendered page:
📚 Documentation preview 📚: https://cpython-previews--118237.org.readthedocs.build/