Skip to content

Feature/Travis CI configuration#104

Merged
jakinyele merged 4 commits intoJHUISI:devfrom
dennisschroer:feature/travis-ci
Jun 21, 2016
Merged

Feature/Travis CI configuration#104
jakinyele merged 4 commits intoJHUISI:devfrom
dennisschroer:feature/travis-ci

Conversation

@dennisschroer
Copy link
Copy Markdown
Contributor

This PR correctly sets up continuous integration with Travis by installing dependencies, building the application and executing the tests. It also adds build status to the README page.

Travis was already enabled (https://travis-ci.org/JHUISI/charm), but not correctly configured resulting in a build failure at all time (by default it attempts to test a ruby application). This also gives the (mostly) wrong indication that PRs are not correct.

@jakinyele jakinyele merged commit d6ddaad into JHUISI:dev Jun 21, 2016
@jakinyele
Copy link
Copy Markdown
Member

Thank you!

@dennisschroer
Copy link
Copy Markdown
Contributor Author

Wow that was quick 👍

@jakinyele
Copy link
Copy Markdown
Member

This has been on my todo list for quite awhile. So, very much appreciated!

@dennisschroer
Copy link
Copy Markdown
Contributor Author

I see that is not quite correct now, https://travis-ci.org/JHUISI/charm/builds/139206275 fails but travis says it is succesful. Let me see if I can fix that

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