-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[BEAM-1251] Upgrade pylint version for py27-lint3 #6489
Conversation
@@ -191,6 +191,7 @@ def expand(self, pcoll): | |||
|
|||
class SafeFastPrimitivesCoder(coders.Coder): | |||
"""This class add an quote/unquote step to escape special characters.""" | |||
# pylint: disable=deprecated-urllib-function |
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 code has already been changed in #6451
@@ -55,6 +55,7 @@ def instance_to_type(o): | |||
if o is None: | |||
return type(None) | |||
elif t not in typehints.DISALLOWED_PRIMITIVE_TYPES: | |||
# pylint: disable=deprecated-types-field |
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 needed for old-style Python 2 classes.
@@ -81,6 +81,7 @@ def __init__(self, side_inputs): | |||
self._transform_to_side_inputs[side.pvalue.producer].append(side) | |||
|
|||
def __repr__(self): | |||
# pylint: disable=dict-values-not-iterating |
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.
Would s/if self._views.values()
/if self._views
/ remove the lint warning?
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.
You're right. I've removed the values()
call.
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.
Thanks a lot, @RobbeSneyders
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.
Awesome!
* Upgrade pylint for py27-lint3 test to 1.9.3
* Upgrade pylint for py27-lint3 test to 1.9.3
@tvalentyn @Fematich @aaltay
Post-Commit Tests Status (on master branch)