-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
bpo-39639: remove suite node #18513
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
bpo-39639: remove suite node #18513
Conversation
61b7b0b
to
1a0874e
Compare
I haven't touched the AST in a while, so it would be great if someone else can just double-check me. Otherwise, @isidentical , if no other review comes in by Friday just ping me and I will merge this. |
Misc/NEWS.d/next/Core and Builtins/2020-02-15-15-29-34.bpo-39639.3mqJjm.rst
Outdated
Show resolved
Hide resolved
Co-Authored-By: Brett Cannon <[email protected]> Co-Authored-By: Pablo Galindo <[email protected]>
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.
I suggest to change the PR title to " bpo-39639: Remove ast.Suite node class". It can be done when the PR is merged.
Co-Authored-By: Victor Stinner <[email protected]>
Misc/NEWS.d/next/Core and Builtins/2020-02-15-15-29-34.bpo-39639.3mqJjm.rst
Outdated
Show resolved
Hide resolved
@isidentical: Thanks for the doc update, it's now well better ;-) Thanks also for clearing the old dust of from the early age of Python ;) |
The AST "Suite" node is no longer used and it can be removed from the ASDL definition and related structures (compiler, visitors, ...). Co-Authored-By: Victor Stinner <[email protected]> Co-authored-by: Brett Cannon <[email protected]> Co-authored-by: Pablo Galindo <[email protected]>
https://bugs.python.org/issue39639