Skip to content

FIX Avoid side effects importing pip/setuptool in show_version#22621

Merged
thomasjpfan merged 2 commits intoscikit-learn:mainfrom
ogrisel:use-importlib.metadata
Feb 28, 2022
Merged

FIX Avoid side effects importing pip/setuptool in show_version#22621
thomasjpfan merged 2 commits intoscikit-learn:mainfrom
ogrisel:use-importlib.metadata

Conversation

@ogrisel
Copy link
Copy Markdown
Member

@ogrisel ogrisel commented Feb 26, 2022

Fixes #22614.
Requires dropping Python 3.7 support first: #22617.

@ogrisel
Copy link
Copy Markdown
Member Author

ogrisel commented Feb 26, 2022

I checked that all the CI failures are ModuleNotFoundError: No module named 'importlib.metadata' which will go away once we merge #22617.

@ogrisel ogrisel added the Quick Review For PRs that are quick to review label Feb 26, 2022
Copy link
Copy Markdown
Member

@jeremiedbb jeremiedbb left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Member

@thomasjpfan thomasjpfan left a comment

Choose a reason for hiding this comment

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

LGTM

@thomasjpfan thomasjpfan changed the title FIX avoid side effect of importing pip/setuptools in show_versions() FIX Avoid side effects importing pip/setuptool in show_version Feb 28, 2022
@thomasjpfan thomasjpfan merged commit 817f069 into scikit-learn:main Feb 28, 2022
jeremiedbb added a commit to jeremiedbb/scikit-learn that referenced this pull request Mar 1, 2022
…t-learn#22621)

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>
@ogrisel ogrisel deleted the use-importlib.metadata branch March 1, 2022 16:16
thomasjpfan pushed a commit to thomasjpfan/scikit-learn that referenced this pull request Mar 1, 2022
…t-learn#22621)

Co-authored-by: Jérémie du Boisberranger <34657725+jeremiedbb@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:utils Quick Review For PRs that are quick to review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

joblib development version breaks sklearn.show_versions()

3 participants