1092 add guided advanced#1093
Conversation
Code Coverage SummaryDiff against mainResults for commit: 1abc270 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Tests Summary 1 files 30 suites 1m 52s ⏱️ Results for commit 1abc270. ♻️ This comment has been updated with latest results. |
Unit Test Performance Difference
Results for commit 9e4fa14 ♻️ This comment has been updated with latest results. |
Melkiades
left a comment
There was a problem hiding this comment.
nice addition. code works and the substantive vignettes render. a few things:
- NEWS bug-fix line points at
obj_na_str<-but the recycling fix is inindent_mod<-.obj_na_str<-does not recycle (checked, stays length 1) c.RowsVerticalSectionroxygen missing blank line after title- no tests for
c()or the new accessor methods - 6 of the vignettes are stubs, up to you if those go in now
not blocking on the stubs, but i'd want tests and the NEWS fix before merge
| * Added vignette: Guided Tour (Advanced) - (stub) `TableTree` Objects - (stub) Custom Pruning Functions @gmbecker | ||
|
|
||
| ### Bug Fixes | ||
| * `obj_na_str<-` RowsVerticalSection method now correctly recycles length 1 values @gmbecker |
There was a problem hiding this comment.
this is indent_mod<-, not obj_na_str<-. obj_na_str<- does not recycle (checked, stays length 1)
| invisible(x) | ||
| } | ||
|
|
||
| #' Combine RowsVerticalSection objects |
There was a problem hiding this comment.
missing blank line after title, so title and description end up the same text in the .Rd
| #' @returns A single RowsVerticalSection object containing all | ||
| #' row sections from the objects passed to `...` | ||
| #' @export | ||
| c.RowsVerticalSection <- function(...) { |
There was a problem hiding this comment.
no tests for c() or the new RVS accessors. at least combine + error path + na_str/format carryover
closes #1090 and #1092