search

mailgun / talon

Apache License 2.0
1.27k stars 285 forks source link

add travis ci configuration #187

Open marcindulak opened 5 years ago

marcindulak commented 5 years ago

See https://www.youtube.com/watch?v=FEXY1ZP-sBs for and introduction to Travis CI.

The platforms tested: centos7, fedora rawhide, debian latest, several ubuntu images, osx. Python versions tested: python2 and python3.

Note that nosetests fail with python3 (see the failing builds at https://travis-ci.org/marcindulak/talon/builds/488964077 where python3 builds were enabled) and therefore python3 builds are disabled in .travis.yml in this PR. The current status of the builds in this PR (limited to python2) is seen on https://travis-ci.org/marcindulak/talon/builds/546206473

Python3 support needs to be improved https://github.com/mailgun/talon/issues/42 https://github.com/mailgun/talon/pull/136, a separate bug can be opened after confirming that the tests fail with python3.

This PR adds also requirements-dev.txt file to make the builds with support for testing easier, this is related to https://github.com/mailgun/talon/issues/167

The change in setup.py contained in this PR is needed in order for python3 to read README.rst (see e.g. https://github.com/enthought/mayavi/issues/713)

I'm reading that the migration to Travis CI was planned https://github.com/mailgun/talon/pull/166

mailgun-ci commented 5 years ago

Can one of the admins verify this patch?

marcindulak commented 5 years ago

@obukhov-sergey - are you still considering travis?