-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
[Dy2Stat] Remove gast.Index for compatibility of gast 0.4.0 #31358
Conversation
Thanks for your contribution! |
✅ This PR's description meets the template requirements! |
a9498f0
to
d11c7ae
Compare
11fa61f
to
d8ab03e
Compare
e7bcfac
to
0cc4f4b
Compare
0cc4f4b
to
8d5c843
Compare
8d5c843
to
076bbe3
Compare
076bbe3
to
d63d7d8
Compare
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.
LGTM
d63d7d8
to
827e5b2
Compare
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.
LGTM
PR types
Bug fixes
PR changes
Others
Describe
As the title
When gast>=0.4.0, gast.Index is not used, which is replaced other gast node such as gast.Constant, gast.Name, gast.UnaryOp and so on.
Considering the compatibility of gast, here use ast note to check whether the node is a num. For more details, please visit https://github.com/serge-sans-paille/gast