Skip to content
This repository was archived by the owner on Dec 6, 2023. It is now read-only.

[MRG] Continuous integration on Python 3.6 - #118

Closed
vene wants to merge 3 commits into
scikit-learn-contrib:masterfrom
vene:bumpci
Closed

[MRG] Continuous integration on Python 3.6#118
vene wants to merge 3 commits into
scikit-learn-contrib:masterfrom
vene:bumpci

Conversation

@vene

@vene vene commented Jun 10, 2017

Copy link
Copy Markdown
Contributor

In appveyor I am a bit confused because the PYTHON env var seems to point to "system" python while MINICONDA points to a conda-installed python, and these are different in the provided build environment, as shown here. Let's see if this works right.

@vene vene changed the title Continuous integration on Python 3.6 [MRG] Continuous integration on Python 3.6 Jun 10, 2017
Comment thread .travis.yml
language: python
python:
- "2.7"
- "3.4"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

up for debate :P

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.

Fine for me

@vene

vene commented Jun 10, 2017

Copy link
Copy Markdown
Contributor Author

we do not seem to need conda-build in appveyor.yml

The 64-bit py36 failure in the debiasing test seems genuine and very, very weird. (see #117)

@fabianp

fabianp commented Jun 10, 2017

Copy link
Copy Markdown
Member

Feel free to remove conda-build, it probably comes from the time (pre conda-forge) where I used this to build the conda package

@vene

vene commented Jun 10, 2017

Copy link
Copy Markdown
Contributor Author

it would be nice to figure out the reason for the weird failure and fix it... I'm clueless

@StrikerRUS

Copy link
Copy Markdown
Contributor

With merged #161 this PR can be closed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants