-
-
Notifications
You must be signed in to change notification settings - Fork 275
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
Bump wrapt
to 1.14
#1745
Bump wrapt
to 1.14
#1745
Conversation
Pull Request Test Coverage Report for Build 2904545100
💛 - Coveralls |
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.
I think we should differentiate between Python 3.11 and other interpreters. We are compatible with wrap lower than 1.14 after all, let's not create dependency issues fort user if we can easily avoid them.
But |
The issue is not the version of the interpreter but the version required by all the other libs that can be installed by someone wanting to use pylint. There's probably a dependency on pypi or in an internal pypi repository somewhere that ask for wrapt < 1.14 (maybe for good reason, maybe without good reason but...). pylint/astroid would be impossible to install alongside it. |
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.
The diff looks bad, but I don't know what I did wrong.
Co-authored-by: Pierre Sassoulas <pierre.sassoulas@gmail.com>
Steps
Description
Ref. #1516 (comment).
Type of Changes
Related Issue