hrbrmstr / freebase

👃🏽A 'usethis'-like Package for Base R Pseudo-equivalents of 'tidyverse' Code
https://hrbrmstr.github.io/freebase/index.html
Other
83 stars 7 forks source link

inst versions are saved without roxygen2 comments #5

Open yonicd opened 6 years ago

yonicd commented 6 years ago

when invoking use* the script that is copied to R doesnt contain comments that fit with roxygen2. i think the script in inst should have comments that are roxygen2 ready.

hrbrmstr commented 6 years ago

so mebbe have roxygen without the ! and let folks click-drag-insert-!?

yonicd commented 6 years ago

without the !?, where do you see !? why does regular #' bother in inst?

hrbrmstr commented 6 years ago

oh i meant ' (mis-hit due to multi-tasking).

It might not bother in inst.

Hrm. If it doesn't have an #' @export and has a #' noRd it's likely fully benign to pkg authors too

yonicd commented 6 years ago

from my build/install it didnt bother the package bulid that valid roxygen2 (even w @export and @rdname are there.