-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set up CI #1768
Comments
We actually already have CI set up at http://robot.radiaatto.ri.fi but it hasn't been updated yet to track GitHub repository. It runs tests on Python 2.5-2.7, Jython 2.5 and IronPython 2.7, both on Linux and Windows. If we get same platform support elsewhere, I'd be happy to swtich. Otherwise we just need to reconfigure old service and also add a note about it to README and/or other docs. Definitely a must for 2.8.6. |
Oh, cool, I never saw this. Jenkins has the capability to build GitHub pull requests, too, though it requires a plugin. https://wiki.jenkins-ci.org/display/JENKINS/GitHub+pull+request+builder+plugin |
I think Travis is capable of running IronPython and Jython if configured properly, but if you already have Jenkins set up then I don't see a good reason to switch. Example Travis config with IronPython: https://github.com/earthreader/libearth/blob/master/.travis.yml |
Turns out Travis phased out support for Python 2.5, so Jython testing isn't really possible. I suggest sticking with the existing Jenkins setup. |
I just updated the jobs to poll now the github project. So that should work. I will next take a look at getting the automatic pullrequest testing to work. (Or semiautomatic, we need to manually check the pull request before accepting it to run on the CI machine of course.) |
Now there is an almost automatic build for pull requests. We still need figure where to document the ci. Probably to contribution instructions. |
I agree that CONTRIBUTING.rst is the best place to write information about CI, but the problem is that we don't yet have it. I think this is what we should do:
|
You can pretty easily set up a job that will automatically run tests against every commit and every pull request. I can help with this if there's interest.
For example, see https://travis-ci.org/guykisel/robotframework-faker
The text was updated successfully, but these errors were encountered: