Merge pull request #11 from dynverse/devel

various travis fixes

Author: rcannood

.travis.yml

@@ -1,11 +1,14 @@
 language: r
 sudo: required
 dist: trusty
+os:
+ - linux
+ - osx
 cache: packages
 install:
- - if [ `git branch | grep '* master' | wc -l` == 0 ]; then sed -i 's#\(dynverse/[^, ]*\)#\1@devel#' DESCRIPTION; fi
+ - if [[ "$TRAVIS_OS_NAME" == "osx" ]]; then brew install libgit2; fi
  - R -e 'if ("devtools" %in% rownames(installed.packages())) update.packages(oldPkgs = "devtools", ask = FALSE) else install.packages("devtools")'
  - R -e 'devtools::install_cran("covr")'
- - R -e 'setRepositories(ind = 1:4); devtools::install(dependencies = T, upgrade_dependencies = T)'
+ - R -e 'setRepositories(ind = 1:4); devtools::install(dependencies = TRUE, upgrade_dependencies = TRUE)'
 after_success:
  - R -e 'covr::codecov()'

cran-comments.md

@@ -3,6 +3,8 @@
 dynutils 1.0.0 comes with a lot of new functionality,
 improved documentation and better tests.
 
+The notes and errors from the previous release has been fixed.
+
   * REMOVED `run_until_exit()`: This functionality is already 
     sufficiently covered by the `processx` package.
 
@@ -41,10 +43,6 @@ I'm not sure what specifically triggers the note message.
 
 ## Reverse dependencies
 
-This is a new release, so there are no reverse dependencies.
-
----
-
 * I have run R CMD check on the 1 downstream dependencies.
   (Summary at [revdep/README.md](revdep/README.md)).