updated CRAN comments

beanumber · Nov 29, 2016 · a96c2f6 · a96c2f6
1 parent dec8640
commit a96c2f6
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 5 deletions.
diff --git a/R/etl_create.R b/R/etl_create.R
@@ -42,8 +42,9 @@
 #'   db <- src_postgres(dbname = "mtcars", user = "postgres", host = "localhost")
 #'   cars <- etl("mtcars", db)
 #' }
-#' if (require(RMySQL)) {
-#'   db <- src_mysql(dbname = "mtcars", user = "r-user", host = "localhost", password = "mypass")
+#' if (require(RMySQL) && mysqlHasDefault()) {
+#'   db <- src_mysql(dbname = "mtcars", user = "r-user",
+#'                   host = "localhost", password = "mypass")
 #'   cars <- etl("mtcars", db)
 #' }
 #' }

diff --git a/cran-comments.md b/cran-comments.md
@@ -9,7 +9,8 @@
 0 errors | 0 warnings | 0 notes
 
 * Code coverage is 85%.
-* I believe this will fix the error reported here (https://www.r-project.org/nosvn/R.check/r-devel-linux-x86_64-fedora-clang/etl-00check.html)
+* I believe this will fix the error reported here (https://www.r-project.org/nosvn/R.check/r-devel-linux-x86_64-fedora-clang/etl-00check.html). I checked this
+with r-hub on Fedora and found no errors. 
 
 ## Reverse dependencies
 

diff --git a/man/etl_create.Rd b/man/etl_create.Rd