-
Notifications
You must be signed in to change notification settings - Fork 90
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
Remove warnings created by version checks #261
Remove warnings created by version checks #261
Conversation
I'm not very happy about adding a requirement, I must say. Every requirement is an additional liability and amount of work for a user. |
…fix_distutil_deprecation � Conflicts: � dirty_cat/super_vectorizer.py
Following the previous point, I've added a little replacement class that works pretty much like the two other packages. |
I like the idea, but we have a lot of failing tests right now :) |
…fix_distutil_deprecation � Conflicts: � dirty_cat/gap_encoder.py � dirty_cat/similarity_encoder.py
Should be good now ! |
That's great! Thanks a lot. Merging! |
Fixes #260 by replacing
distutils.version.LooseVersion
bypackaging.version.Version
.Therefore, it also adds
packaging
to the requirements.