diff --git a/cran-comments.md b/cran-comments.md index 4124615c..8d6a06a1 100644 --- a/cran-comments.md +++ b/cran-comments.md @@ -1,37 +1,28 @@ # Submission -2018-03-21 +2018-07-17 ## Submission summary -An overhaul of core package functions, including breaking changes. Brings the package to a stable long-term state, as reflected by the 1.0.0 version number. +Per CRAN request (Kurt Hornik on July 3rd 2018), I am submitting a patched version that addresses two dependency-based warnings currently being thrown under R-devel. There are a few minor changes to the package functionality as well. ### Test environments #### Windows -* local Windows 10 install, R 3.4.3 -* local Windows 10 install, R 3.4.4 -* win-builder with R-devel 2018-03-20 r74431 +* win-builder with R Under development (unstable) (2018-07-15 r74966) #### Linux -* ubuntu 16.04.3, R-release 3.4.3 (local) -* ubuntu 16.04.3, R-release 3.4.4 (local) -* ubuntu 14.04.5, R-oldrel 3.3.3 (Travis CI) -* ubuntu 14.04.5 R-release 3.4.4 (Travis CI) -* ubuntu 14.04.5 R-devel R 3.5.0 74418 (Travis CI) +* ubuntu 16.04.3, R-oldrel 3.4.4 (local) +* ubuntu 14.04.5, R-oldrel 3.4.4 (Travis CI) +* ubuntu 14.04.5 R-release 3.5.0 (Travis CI) +* ubuntu 14.04.5 R-devel R Under development (unstable) (2018-06-20 r74923) (Travis CI) #### Mac -* OS X Sierra 10.12.6, R-oldrel 3.3.3 (Travis CI) -* OS X Sierra 10.12.6, R-release 3.4.4 (Travis CI) +* OS X Sierra 10.12.6, R-oldrel 3.4.4 (Travis CI) +* OS X Sierra 10.12.6, R-release 3.5.0 (Travis CI) ### R CMD check results 0 errors | 0 warnings | 0 notes ### Downstream dependencies -The janitor v1.0 update involves breaking changes that affect some downstream packages. I advised all downstream package maintainers of these changes on February 21, including providing example code that would keep their packages compatible with all versions of janitor. All downstream packages have since been updated to address the janitor 1.0.0 changes. +The only breaking change in this release that could affect another package is quite obscure and does not affect the downstream dependencies. -I checked 4 reverse dependencies from CRAN. - -#### New problems -These CRAN packages depending on janitor each have 1 warning: - -* ballr, bomrang: failures due to change in janitor::clean_names. I emailed the maintainers notice of failing R CMD check on Mar 17. Maintainers of both packages have since confirmed to me that they've updated their packages accordingly. Both packages have been submitted to CRAN or will be shortly following janitor 1.0.0's acceptance. -* fivethirtyeight: my check yielded a warning `checking for unstated dependencies in example ... WARNING warning in parse(file = files, n = -1L) : invalid input found on input connection 'fivethirtyeight-Ex.R'.` I discussed this warning with the maintainer; he tested janitor 1.0.0 with his package and confirmed that any WARNING or NOTE is not the result of changes to janitor. +I checked 5 reverse dependencies from CRAN: ballr, bomrang, driftR, fivethirtyeight, moderndive. I get some unrelated warnings and notes but nothing having to do with the janitor package or this update. \ No newline at end of file diff --git a/revdep/README.md b/revdep/README.md index 463c0b41..d3dadf18 100644 --- a/revdep/README.md +++ b/revdep/README.md @@ -2,61 +2,54 @@ |field |value | |:--------|:----------------------------| -|version |R version 3.4.3 (2017-11-30) | +|version |R version 3.4.4 (2018-03-15) | |os |Ubuntu 16.04.3 LTS | |system |x86_64, linux-gnu | |ui |RStudio | |language |en_US | |collate |en_US.UTF-8 | -|tz |posixrules | -|date |2018-03-17 | +|tz |America/New_York | +|date |2018-07-17 | # Dependencies |package |old |new |Δ | |:----------|:--------|:--------|:--| -|janitor |0.3.1 |1.0.0 |* | +|janitor |1.0.0 |1.1 |* | |assertthat |0.2.0 |0.2.0 | | |BH |1.66.0-1 |1.66.0-1 | | |bindr |0.1.1 |0.1.1 | | -|bindrcpp |0.2 |0.2 | | +|bindrcpp |0.2.2 |0.2.2 | | |cli |1.0.0 |1.0.0 | | |crayon |1.3.4 |1.3.4 | | -|dplyr |0.7.4 |0.7.4 | | +|dplyr |0.7.6 |0.7.6 | | +|fansi |0.2.3 |0.2.3 | | |glue |1.2.0 |1.2.0 | | |magrittr |1.5 |1.5 | | -|pillar |1.2.1 |1.2.1 | | +|pillar |1.3.0 |1.3.0 | | |pkgconfig |2.0.1 |2.0.1 | | -|plogr |0.1-1 |0.1-1 | | -|purrr |0.2.4 |0.2.4 | | +|plogr |0.2.0 |0.2.0 | | +|purrr |0.2.5 |0.2.5 | | |R6 |2.2.2 |2.2.2 | | -|Rcpp |0.12.16 |0.12.16 | | -|rlang |0.2.0 |0.2.0 | | -|snakecase |NA |0.9.0 |* | -|stringi |1.1.7 |1.1.7 | | -|stringr |NA |1.3.0 |* | +|Rcpp |0.12.17 |0.12.17 | | +|rlang |0.2.1 |0.2.1 | | +|snakecase |0.9.1 |0.9.1 | | +|stringi |1.2.3 |1.2.3 | | +|stringr |1.3.1 |1.3.1 | | |tibble |1.4.2 |1.4.2 | | -|tidyr |0.8.0 |0.8.0 | | +|tidyr |0.8.1 |0.8.1 | | |tidyselect |0.2.4 |0.2.4 | | -|utf8 |1.1.3 |1.1.3 | | +|utf8 |1.1.4 |1.1.4 | | # Revdeps -## Couldn't check (4) +## All (5) -|package |version |error |warning |note | -|:---------------|:-------|:-----|:-------|:----| -|ballr |? | | | | -|bomrang |? | | | | -|fivethirtyeight |? | | | | -|moderndive |? | | | | - -## All (4) - -|package |version |error |warning |note | -|:---------------|:-------|:-----|:-------|:----| -|ballr |? | | | | -|bomrang |? | | | | -|fivethirtyeight |? | | | | -|moderndive |? | | | | +|package |version |error |warning |note | +|:----------------------------------------------|:-------|:-----|:-------|:----| +|[ballr](problems.md#ballr) |0.2.1 |1 |1 | | +|[bomrang](problems.md#bomrang) |0.2.1 |1 | | | +|[driftR](problems.md#driftr) |1.1.0 |1 |1 | | +|[fivethirtyeight](problems.md#fivethirtyeight) |0.4.0 |1 |1 |3 | +|[moderndive](problems.md#moderndive) |0.2.0 |1 |1 | | diff --git a/revdep/problems.md b/revdep/problems.md index e69de29b..028bf852 100644 --- a/revdep/problems.md +++ b/revdep/problems.md @@ -0,0 +1,107 @@ +# ballr + +Version: 0.2.1 + +## In both + +* checking PDF version of manual without hyperrefs or index ... ERROR + ``` + Re-running with no redirection of stdout/stderr. + Hmm ... looks like a package + You may want to clean up by 'rm -rf /tmp/Rtmp7Fl3iL/Rd2pdf7d8e560b6b41' + ``` + +* checking PDF version of manual ... WARNING + ``` + LaTeX errors when creating PDF version. + This typically indicates Rd problems. + ``` + +# bomrang + +Version: 0.2.1 + +## In both + +* checking package dependencies ... ERROR + ``` + Package required but not available: ‘rgdal’ + + See section ‘The DESCRIPTION file’ in the ‘Writing R Extensions’ + manual. + ``` + +# driftR + +Version: 1.1.0 + +## In both + +* checking PDF version of manual without hyperrefs or index ... ERROR + ``` + Re-running with no redirection of stdout/stderr. + Hmm ... looks like a package + You may want to clean up by 'rm -rf /tmp/RtmpDl2GqW/Rd2pdf174f4745dddc' + ``` + +* checking PDF version of manual ... WARNING + ``` + LaTeX errors when creating PDF version. + This typically indicates Rd problems. + ``` + +# fivethirtyeight + +Version: 0.4.0 + +## In both + +* checking PDF version of manual without hyperrefs or index ... ERROR + ``` + Re-running with no redirection of stdout/stderr. + Hmm ... looks like a package + You may want to clean up by 'rm -rf /tmp/RtmpA7JSN5/Rd2pdf3c0733592067' + ``` + +* checking PDF version of manual ... WARNING + ``` + LaTeX errors when creating PDF version. + This typically indicates Rd problems. + ``` + +* checking package dependencies ... NOTE + ``` + Package suggested but not available for checking: ‘fivethirtyeight’ + ``` + +* checking installed package size ... NOTE + ``` + installed size is 6.6Mb + sub-directories of 1Mb or more: + data 5.4Mb + ``` + +* checking data for non-ASCII characters ... NOTE + ``` + Note: found 1616 marked UTF-8 strings + ``` + +# moderndive + +Version: 0.2.0 + +## In both + +* checking PDF version of manual without hyperrefs or index ... ERROR + ``` + Re-running with no redirection of stdout/stderr. + Hmm ... looks like a package + You may want to clean up by 'rm -rf /tmp/Rtmpw8bhRh/Rd2pdf48086135c2d3' + ``` + +* checking PDF version of manual ... WARNING + ``` + LaTeX errors when creating PDF version. + This typically indicates Rd problems. + ``` +