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
[WIP] Upgrade pylint to 1.5.x #2249
Conversation
… dicts and lists. This way pylint detection can be smarter since it's also aware of the attribute type.
datetime.datetime objects look like.
# Hack for attributes with multiple types (e.g. string, null) | ||
property_type = property_type[0] | ||
|
||
if property_type == 'object': |
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.
This is cool since pylint is now also aware of each API model attribute type.
It means we get improved static checking for API model members (e.g. if API model attribute type is defined as string and you want perform division on it, pylint will now throw since it knows the attribute type is string a not int / float).
I decided to wait with upgrade to pylint 1.5.0 since I found some more blocking regressions in pylint. I will cherry-pick other improvements in a new branch and get back to this branch once new version of pylint with fixes is released. |
Superseded by #2613. |
This pull request upgrades pylint and astroid to latest version which include some checking related improvements.
In addition that, it includes some lint cleanup and related improvements.