mattjj / pyhsmm

MIT License
546 stars 173 forks source link

travis CI #36

Closed mattjj closed 8 years ago

mattjj commented 9 years ago

Inspired by the recent successful setup.py improvements and pypi integration, it would be great to get pyhsmm and pybasicbayes tests set up with Travis CI.

yarden commented 9 years ago

Integration with travis CI would be great. This might require one last upgrade to setup.py to force the CI system to run Cython when making a source distribution (python setup.py sdist), according to this: http://stackoverflow.com/questions/4505747/how-should-i-structure-a-python-package-that-contains-cython-code

Based on that post, we'd need to override the sdist class using something like this code (from above link):

from distutils.command.sdist import sdist as _sdist
class sdist(_sdist):
    def run(self):
        # Make sure the compiled Cython files in the distribution are up-to-date
        from Cython.Build import cythonize
        cythonize(['cython/mycythonmodule.pyx'])
        _sdist.run(self)
cmdclass['sdist'] = sdist

I don't have experience with travis CI so I don't know if that'd be necessary.

mattjj commented 8 years ago

Just added Travis integration!