PR: Add appveyor CI integration

[goanpeca]

Fixes #4

[ccordoba12]

You should use CMD_IN_ENV here. Please take a look at the Spyder yaml file.

[goanpeca]

yes!

[goanpeca]

@andfoy test are running but I am not getting any coverage :-|

Also I could not run with python setup.py develop/install so I built the recipe (with conda build) and then installed that one and run pytest from that installed one.

Thoughts?

[andfoy]

Actually, we're missing some env variables to compile using setuptools, I will add them. By the way, our python packages are not including Cython sources (pyx files), only C transpiled files. Should we include pyx files by default? @goanpeca that's why coverage is not reported when you use conda-build

[goanpeca]

Actually, we're missing some env variables to compile using setuptools, I will add them. By the way, our python packages are not including Cython sources (pyx files), only C transpiled files. Should we include pyx files by default? @goanpeca that's why coverage is not reported when you use conda-build

Yes we should...

@andfoy

[goanpeca]

Also that last test seems to need to be tweaked cause now it is failing

[goanpeca]

@andfoy if you can paste as a message here the env variables needed to use setup tools, I can add them to the process.

And yes I think we need to bundle the cython sources, not only the compiled c ones

[andfoy]

@goanpeca You need to set LIBRARY_INC=C:\<anaconda path>\envs <If there is an env>\Library\include and LIBRARY_LIB=C:\<anaconda path>\envs <If there is an env>\Library\lib

[andfoy]

Also, you need to compile Cython sources inplace before testing: python setup.py build_ext -i --compiler=msvc

[andfoy]

Do not forget also to set DISTUTILS_USE_SDK=1

[andfoy]

And yes I think we need to bundle the cython sources, not only the compiled c ones

I will open an issue to add PYX sources to the MANIFEST.in

[goanpeca]

@andfoy actually, you need to update that on the readme... or create a BUILD.md, to give conda instructions (one line!) vs the other haha.

I prefer to use conda-build in that case, @andfoy you mean that if I include the cython sources, with MANIFEST.in it should work?

[andfoy]

I prefer to use conda-build in that case, @andfoy you mean that if I include the cython sources, with MANIFEST.in it should work?

Let me confirm it!

[andfoy]

@andfoy actually, you need to update that on the readme... or create a BUILD.md, to give conda instructions (one line!) vs the other haha.

I will add them, I actually forgot completely about all those variables haha

[andfoy]

By the way, do not forget to set CYTHON_COVERAGE=1

[goanpeca]

Yep that was the one I did set up

[andfoy]

This command does not install nothing at all to site-packages, it only compiles all files locally

[goanpeca]

Yep I just realized XD

[andfoy]

Call? Haha

[goanpeca]

yes.... :-p

[goanpeca]

Ok @andfoy and @ccordoba12 this is ready, great work @andfoy :-)

I need to fix something on coveralls-python, but that is a separate issue

[andfoy]

Thanks for the great effort! @goanpeca

[ccordoba12]

Great work @goanpeca!

[andfoy]

Should I merge this? @goanpeca