Skip to content
This repository was archived by the owner on Mar 1, 2018. It is now read-only.

Adding .travis.yml in prep for adding TravisCI#183

Merged
jnraine merged 1 commit into
masterfrom
travis
Jun 8, 2013
Merged

Adding .travis.yml in prep for adding TravisCI#183
jnraine merged 1 commit into
masterfrom
travis

Conversation

@rickychilcott

Copy link
Copy Markdown
Contributor

@jnraine can you add the appropriate configuration for TravisCI? http://about.travis-ci.org/docs/user/getting-started/#Step-one%3A-Sign-in

jnraine added a commit that referenced this pull request Jun 8, 2013
Adding .travis.yml in prep for adding TravisCI
@jnraine jnraine merged commit aafdff1 into master Jun 8, 2013
@jnraine

jnraine commented Jun 8, 2013

Copy link
Copy Markdown
Owner

Done!

@jnraine

jnraine commented Jun 8, 2013

Copy link
Copy Markdown
Owner

@rickychilcott looks like this needs more config? I haven't used travis before and this project isn't test-heavy. Do you have more insight into making this build pass?

One thing I found after a quick google: http://stackoverflow.com/questions/9321153/rails-database-setup-on-travis-ci

@rickychilcott

Copy link
Copy Markdown
Contributor Author

Yeah.. I'll take a look at that StackOverflow and add the appropriate files early this week. I think we should be adding tests for new functionality, where possible.

It's failing for me locally too, some of the Nokogiri code is throwing errors, so maybe you could look at that when you get a chance too.

Ricky Chilcott
ricky@rbsavvy.com

On Jun 8, 2013, at 3:15 PM, Jordan Raine notifications@github.com wrote:

@rickychilcott looks like this needs more config? I haven't used travis before and this project isn't test-heavy. Do you have more insight into making this build pass?

One thing I found after a quick google: http://stackoverflow.com/questions/9321153/rails-database-setup-on-travis-ci


Reply to this email directly or view it on GitHub.

@rickychilcott

Copy link
Copy Markdown
Contributor Author

@jnraine Got it to finally run: https://travis-ci.org/jnraine/munkiserver/jobs/7916587

If you can help get those tests to pass, cool, if not, let me know. Also code coverage is ~45%, which isn't really bad. Seems high to me, but… kinda neat!

On Jun 8, 2013, at 3:15 PM, Jordan Raine notifications@github.com wrote:

@rickychilcott looks like this needs more config? I haven't used travis before and this project isn't test-heavy. Do you have more insight into making this build pass?

One thing I found after a quick google: http://stackoverflow.com/questions/9321153/rails-database-setup-on-travis-ci


Reply to this email directly or view it on GitHub.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants