Skip to content

Bump minimal requirement for sklearn from 0.20.0 to 0.21.0 - #202

Merged
LilianBoulard merged 2 commits into
skrub-data:masterfrom
LilianBoulard:bump_sklearn_to_v021
Jul 23, 2021
Merged

LilianBoulard merged 2 commits into
skrub-data:masterfrom
LilianBoulard:bump_sklearn_to_v021

Conversation

@LilianBoulard

@LilianBoulard LilianBoulard commented Jul 23, 2021 •

Copy link
Copy Markdown
Member

Necessary for #155 and #200

Approved by @GaelVaroquaux

Comment thread CHANGES.rst
Release 0.2.0
=============

Also see pre-release 0.2.0a1 below for additional changes.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I wonder whether we should separate the alpha from the final?

@GaelVaroquaux

Copy link
Copy Markdown
Member

I think that we need to change the version in the setup.py too.

@LilianBoulard

LilianBoulard commented Jul 23, 2021 •

Copy link
Copy Markdown
Member Author

I believe we're good to go, merging !

@LilianBoulard
LilianBoulard merged commit 814e7cd into skrub-data:master Jul 23, 2021
@LilianBoulard
LilianBoulard deleted the bump_sklearn_to_v021 branch July 23, 2021 14:52
@LilianBoulard LilianBoulard mentioned this pull request Aug 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants