forked from tidyverse/dplyr
-
Notifications
You must be signed in to change notification settings - Fork 0
/
rcmd-notes.Rmd
26 lines (17 loc) · 1.16 KB
/
rcmd-notes.Rmd
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
The following notes were generated across my local OS X install, ubuntu running on travis-ci and win builder. Response to NOTEs across three platforms below.
* checking dependencies in R code ... NOTE
Namespaces in Imports field not imported from:
'Lahman' 'hflights' 'magrittr' 'methods'
I'm resonably certain this is a spurious warning. Lahman and hflights
are data packages and hence have nothing to import. Functions from
magrittr and methods are used with :: (but during package build
so I don't think the current checks see them)
Checking all dependencies (summary at https://github.com/wch/checkresults/blob/master/dplyr/r-release/00check-summary.txt), discovered one problem, with the freqweights package). I have corresponded with the maintainer, and he will release a new version once dplyr 0.2 is on CRAN.
Since the previous submission, I have fixed the three reported problems:
* Used `packageStartupMessage()` in more places.
* Informed the Rcpp maintainers about
"/home/ripley/R/test-3.2/Rcpp/include/Rcpp/DataFrame.h:55:9: warning:
no return statement in function returning non-void [-Wreturn-type]"
* Fixed a NAMESPACE problem.
Regards,
Hadley