Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

point to caret GitHub repo #556

Closed
wants to merge 3 commits into from
Closed

point to caret GitHub repo #556

wants to merge 3 commits into from

Conversation

znmeb
Copy link
Contributor

@znmeb znmeb commented Dec 21, 2016

The "github-repo" tag in index.Rmd needs to point to the caret repo, not bookdown-demo

@coveralls
Copy link

coveralls commented Dec 21, 2016

Coverage Status

Coverage remained the same at 16.514% when pulling b6b7ede on znmeb:master into 4b5f580 on topepo:master.

@coveralls
Copy link

coveralls commented Dec 22, 2016

Coverage Status

Coverage remained the same at 16.514% when pulling 84f0f0b on znmeb:master into 4b5f580 on topepo:master.

@coveralls
Copy link

coveralls commented Dec 22, 2016

Coverage Status

Coverage remained the same at 16.514% when pulling bca4cf0 on znmeb:master into 4b5f580 on topepo:master.

@znmeb
Copy link
Contributor Author

znmeb commented Dec 22, 2016

More fixes:

  1. type = "both" doesn't work on Linux. See https://github.com/znmeb/caret/issues/1. I've added guards to call installers without type = "both" on Linux.
  2. The loop to install missing packages at the end doesn't find a number of the packages because they're in BioConductor, not CRAN. So I use biocLite in the loop instead.

Note also that the fresh switch is used before it's defined. I didn't change that; I set fresh to TRUE before running the script.

@znmeb znmeb closed this Dec 22, 2016
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.

None yet

2 participants