-
Notifications
You must be signed in to change notification settings - Fork 87
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
Address ast
deprecation warnings
#136
Conversation
7bbe530
to
17ba4fc
Compare
17ba4fc
to
530ea95
Compare
530ea95
to
a6141a8
Compare
ast
deprecation warningsast
deprecation warnings
The CodeFactor issues seem to be unrelated? |
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.
Looks good!
@danthedeckie Please review this PR and merge -- it should resolve newer Python ast
warnings.
All tests (except for Python 2.7) are 🟢 over on my fork: https://github.com/edgarrmondragon/simpleeval/actions/runs/9782703954 ![]() |
Ping @danthedeckie |
Thanks @edgarrmondragon ! I've merged this via #146 , using a slightly different method now that it makes sense to drop old old pythons. Thank you for helping push this all forward. |
Thank you @danthedeckie! |
Description
ast.Num
,ast.Str
andast.NameConstant
continue-on-error: true
for 2.7 cause it's no longer available with actions/setup-pythonPre-approval checklist (for submitter)
Please complete these steps