Travis CI support #174

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

krlmlr commented Oct 24, 2013

via r-travis

Owner

hadley commented Oct 25, 2013

Thanks, but I'm using https://github.com/craigcitro/r-travis for other projects. Would you mind using that instead?

Contributor

krlmlr commented Oct 25, 2013

Thank you. I wasn't aware of the other R-Travis. Currently, my version has some advantages:

  • Zero configuration
  • Checks against both R-stable and R-devel
Owner

hadley commented Oct 25, 2013

Craig's version has the advantage of already being familiar to me, and he's planning on submitting it to travis to add an official R backend. I'd really like to see everyone standardising on one approach.

.travis.yml
+branches:
+ only:
+ - master
+ - /^.*-ci$/
@krlmlr

krlmlr Oct 29, 2013

Contributor

I think this is a good compromise -- testing is enabled when the branch has suffix -ci.

Owner

hadley commented Oct 29, 2013

I think that's a reasonable compromise. Please squash and I'll merge. (Also, if you're going to bump the version in description, please also add a heading the news)

Contributor

krlmlr commented Oct 29, 2013

Done.

Contributor

krlmlr commented Oct 29, 2013

Note how this PR doesn't show the Travis build state anymore, just because it hasn't been built on Travis. See e.g. yihui/knitr#642 for comparison. Do you really want to abandon this feature?

Owner

hadley commented Oct 29, 2013

Ok, that's a bummer. Let's flip and explicitly opt out - how about using the suffix -expt (i.e. for experimental) to opt out of automatic builds.

Contributor

krlmlr commented Oct 29, 2013

Done. As of today, the builds seem to err when trying to install r-base-dev: https://travis-ci.org/krlmlr/plyr/builds . This doesn't happen with knitr builds: https://travis-ci.org/krlmlr/knitr/builds .

enable continuous integration via craigcitro/r-travis
also:

- ignore .Rhistory file created by RStudio
- bump version, add heading for new version number
Contributor

krlmlr commented Oct 30, 2013

Build passes again after recent changes to r-travis.

krlmlr referenced this pull request in krlmlr/r-travis Oct 30, 2013

@krlmlr krlmlr referenced this pull request Nov 4, 2013

Closed

Enhancements for *dply #177

Owner

hadley commented Jan 2, 2014

I guess I did this by hand? Please resubmit if you think you could improve the existing config.

@hadley hadley closed this Jan 2, 2014

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