Skip to content

Add travis ci#1

Merged
thoni56 merged 3 commits intocgreen-devs:masterfrom
d-meiser:add-travis-ci
Sep 2, 2015
Merged

Add travis ci#1
thoni56 merged 3 commits intocgreen-devs:masterfrom
d-meiser:add-travis-ci

Conversation

@d-meiser
Copy link
Contributor

@d-meiser d-meiser commented Sep 1, 2015

This pull request adds a simple configuration for travis continuous integration testing. The test matrix contains shared and static builds with clang and gcc. See https://travis-ci.org/d-meiser/cgreen/builds/78221792 for an example.

The cgreen-devs organization will have to enable the cgreen repo for testing on the travis ci website for this to work. The link for the build status in the readme is dead now but should work as soon as the first test build for cgreen-devs/cgreen has completed.

@d-meiser d-meiser force-pushed the add-travis-ci branch 2 times, most recently from ffa28cf to 1dfec24 Compare September 2, 2015 03:19
@d-meiser d-meiser mentioned this pull request Sep 2, 2015
thoni56 added a commit that referenced this pull request Sep 2, 2015
@thoni56 thoni56 merged commit dbbab8e into cgreen-devs:master Sep 2, 2015
@d-meiser d-meiser deleted the add-travis-ci branch September 3, 2015 02:21
@d-meiser
Copy link
Contributor Author

d-meiser commented Sep 3, 2015

Hi @thoni56 , thanks for merging the PR. Note that you'll need to enable testing on the travis ci website for the tests to work. Cheers, Dominic

@thoni56
Copy link
Contributor

thoni56 commented Sep 3, 2015

Yes, I got that, just haven't gotten around to it yet.

@thoni56
Copy link
Contributor

thoni56 commented Sep 3, 2015

Enabled, so let's trigger a build, ok to test.

@thoni56
Copy link
Contributor

thoni56 commented Sep 3, 2015

Done, and works: https://travis-ci.org/cgreen-devs/cgreen

@thoni56
Copy link
Contributor

thoni56 commented Sep 16, 2015

@d-meiser How did you manage to get travis builds for your local work? Forked the repo under your user and added it to your Travis user?

@d-meiser
Copy link
Contributor Author

@thoni56 correct, that should work.

Note that it's not strictly local. It's still on github but for my fork.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants