GitHub user PommeVerte opened a pull request:
https://github.com/apache/incubator-tinkerpop/pull/148
initial travis setup
Hey guys,
This isn't so much ready for merge. I just wanted to put it out there for
review and suggestions.
This PR aims at adding travis support. It currently checks:
- `mvn clean install`
- `bin/process-docs.sh --dryRun` (will fail if an error occurs, doesn't
actually check docs)
- `mvn process-resources -Djavadoc` (will fail if an error occurs, doesn't
actually check docs)
And the checks
[pass](https://travis-ci.org/PommeVerte/incubator-tinkerpop/builds/92270450)
and take about 20mn to run. There are however a few issues mentioned bellow.
I also ran into some rat-check issues early on. Excluded the `.travis.yml`
from the checks but I don't know if any work needs to be done with licensing
here? I'm guessing not but shout out if it's required.
Suggestions
=========
- adding support for [coveralls](https://coveralls.io/) could be cool.
(test code coverage reports)
- use `ant` as a basis. I'm working on the assumption that that's what you
guys are using in trying to set jenkins up? It should theoretically be possible
to run the same build in both CI systems.
Issues
=====
- `-DincludeNeo4j` currently fails, not really sure why, will need to
investigate. Waiting to sort this out before running integration tests.
- `osx` building/testing is possible but doesn't support the apt addons (it
doesn't even skip them, it just fails). There's a possible workaround for this.
- Need to add support for `bin/process-docs.sh` if you guys have any setup
scripts for hadoop let me know.
Moving forward
============
If you feel that having these checks systematically run on PRs is
desirable, we can merge this early and bring changes to it as we go.
It will only require a PMC with write rights to this repo to setup the
travis account (or configure it)
If there's more stuff you want to see here let me know.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/PommeVerte/incubator-tinkerpop TRAVIS
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-tinkerpop/pull/148.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #148
----
commit fa62af97c8123270460c80a985e78504ce0229bd
Author: Dylan Millikin <[email protected]>
Date: 2015-11-20T02:05:33Z
initial travis setup
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
