-
Notifications
You must be signed in to change notification settings - Fork 6k
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
mgr/dashboard: upgraded python dev dependencies #26007
Conversation
Thank you for taking care of this, @alfonsomthd! Would you mind splitting this PR into two separate commits? I think it would make sense to have one that updates the versions and configuration of our environment, and then a separate cleanup commit that addresses all of the things that have been reported. |
* pylint, astroid: upgraded only for python 3. Latest stable, compatible even with python 3.7 * pycodestyle: 2.4.0 latest stable (python 2 & 3) Signed-off-by: Alfonso Martínez <almartin@redhat.com>
* Fixed linting/style issues found after upgrading: pylint, astroid, pycodestyle Signed-off-by: Alfonso Martínez <almartin@redhat.com>
87d9688
to
f56d2f6
Compare
@LenzGr Sure. Done. |
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.
+1
@@ -832,6 +832,7 @@ def getargspec(func): | |||
func = func.__wrapped__ | |||
except AttributeError: | |||
pass | |||
# pylint: disable=deprecated-method |
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.
How pluggy
devs dealt with this...
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.
we're very likely going full Python 3 by the end of the year. I'd be ok with a pylint for the moment.
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
pylint, astroid: upgraded only for python 3.
Latest stable, compatible even with python 3.7
pycodestyle: 2.4.0 latest stable (python 2 & 3)
Fixed linting/style issues found after upgrade.
Fixes: https://tracker.ceph.com/issues/37385
Signed-off-by: Alfonso Martínez almartin@redhat.com