Skip to content

MAINT explicit error on unsupported Python versions in setup.py#13272

Merged
glemaitre merged 1 commit intoscikit-learn:masterfrom
ogrisel:check-python-version-setup-py
Feb 26, 2019
Merged

MAINT explicit error on unsupported Python versions in setup.py#13272
glemaitre merged 1 commit intoscikit-learn:masterfrom
ogrisel:check-python-version-setup-py

Conversation

@ogrisel
Copy link
Copy Markdown
Member

@ogrisel ogrisel commented Feb 26, 2019

I noticed the default error message can be confusing for potential new users who want to build from source but did not realize they are running on non-compatible Python version.

Copy link
Copy Markdown
Member

@glemaitre glemaitre left a comment

Choose a reason for hiding this comment

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

LGTM

@glemaitre
Copy link
Copy Markdown
Member

Merging when CI are green.

@glemaitre glemaitre merged commit 8ee1347 into scikit-learn:master Feb 26, 2019
@ogrisel ogrisel deleted the check-python-version-setup-py branch February 26, 2019 13:25
xhluca pushed a commit to xhluca/scikit-learn that referenced this pull request Apr 28, 2019
xhluca pushed a commit to xhluca/scikit-learn that referenced this pull request Apr 28, 2019
xhluca pushed a commit to xhluca/scikit-learn that referenced this pull request Apr 28, 2019
koenvandevelde pushed a commit to koenvandevelde/scikit-learn that referenced this pull request Jul 12, 2019
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.

3 participants