Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove 2.8.0 breaking changes in __pkginfo__ #4400

Conversation

Pierre-Sassoulas
Copy link
Member

There was a problem in the code given in the changelog, and we needed it in a more visible section in What's new

Type
βœ“ πŸ“œ Docs

Related Issue

Closes #4399

@coveralls
Copy link

coveralls commented Apr 25, 2021

Coverage Status

Coverage increased (+0.001%) to 91.635% when pulling fc21e3b on Pierre-Sassoulas:breaking-change-in-packaging into 45b4fbd on PyCQA:master.

@Pierre-Sassoulas Pierre-Sassoulas changed the title Better documentation on the packaging breaking changes Draft: Remove 2.8.0 breaking changes in __pkginfo__ Apr 25, 2021
@Pierre-Sassoulas Pierre-Sassoulas force-pushed the breaking-change-in-packaging branch from f5eb153 to c768e31 Compare April 25, 2021 07:40
@Pierre-Sassoulas Pierre-Sassoulas force-pushed the breaking-change-in-packaging branch from c768e31 to fc21e3b Compare April 25, 2021 07:53
@Pierre-Sassoulas Pierre-Sassoulas changed the title Draft: Remove 2.8.0 breaking changes in __pkginfo__ Remove 2.8.0 breaking changes in __pkginfo__ Apr 25, 2021
@Pierre-Sassoulas Pierre-Sassoulas merged commit af1ac5e into pylint-dev:master Apr 25, 2021
@Pierre-Sassoulas Pierre-Sassoulas deleted the breaking-change-in-packaging branch April 25, 2021 09:37
@Pierre-Sassoulas Pierre-Sassoulas added this to the 2.8.1 milestone Apr 25, 2021
Comment on lines +18 to +19
# Kept for compatibility reason, see https://github.com/PyCQA/pylint/issues/4399
numversion = __version__.split(".")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In <2.8 is was a tuple, not a list, see fbcb27a#diff-42df8bb8e2d7b0a18f0174dcb5f3c60b1cbca5a394c3c7f8c9b33c99fb6cda99L34

list causes the following TypeError in pylint-django (pylint-dev/pylint-django#323)

LOAD_CONFIGURATION_SUPPORTED = pylint.__pkginfo__.numversion >= (2, 3)
TypeError: '>=' not supported between instances of 'list' and 'tuple'

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pylama is complaining about pylint not existing
4 participants