You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It's all or nothing. If we use the new roxygen at all, we have to update all of our existing docs. In that PR I added r6 = FALSE to the RoxygenNote to keep the old behavior for now.
The first disqualifying feature I noticed is the that the new R6 stuff doesn't like how we documented several classes in the same file. It just repeats "Super classes" and "Methods" sections down the page. See Documenting multiple R6 (sub)classes in same file r-lib/roxygen2#961.
roxygen2 version 7.0 adds support for documenting R6 classes, rather than the ad hoc approach we've had to take without it: https://github.com/r-lib/roxygen2/blob/master/vignettes/rd.Rmd#L203
Try it out and see how we like it, and consider refactoring the docs to use it everywhere.
Reporter: Neal Richardson / @nealrichardson
Note: This issue was originally created as ARROW-6792. Please see the migration documentation for further details.
The text was updated successfully, but these errors were encountered: