Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[MRG] ENH Remove six dependency #213

Merged
merged 1 commit into from
Jan 1, 2017
Merged

[MRG] ENH Remove six dependency #213

merged 1 commit into from
Jan 1, 2017

Conversation

glemaitre
Copy link
Member

Reference Issue

What does this implement/fix? Explain your changes.

Use the sklearn externals module instead of six directly.

Any other comments?

@coveralls
Copy link

coveralls commented Jan 1, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.0003%) to 98.895% when pulling 89ab391 on glemaitre:remove_external_dep into 53ee3f6 on scikit-learn-contrib:master.

@glemaitre glemaitre merged commit e0bf83b into scikit-learn-contrib:master Jan 1, 2017
christophe-rannou pushed a commit to christophe-rannou/imbalanced-learn that referenced this pull request Apr 3, 2017
@glemaitre
remove six dependencies (scikit-learn-contrib#213)
e8aac82
glemaitre added a commit to glemaitre/imbalanced-learn that referenced this pull request Jun 15, 2017
@glemaitre
remove six dependencies (scikit-learn-contrib#213)
131054d
glemaitre added a commit to glemaitre/imbalanced-learn that referenced this pull request Jun 15, 2017
@glemaitre
remove six dependencies (scikit-learn-contrib#213)
bd0cf77
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@glemaitre @coveralls