Closed ezraporter closed 7 months ago
I think this looks awesome! One tiny thing: please capitalize the final R in REDCapTidieR. :)
Looks like the failure is a snapshot update which is expected.
The warnings weren't appearing for me but seem to be related to .data
for a few instances I used in clean_redcap_long()
. I think here:
I think this looks awesome! One tiny thing: please capitalize the final R in REDCapTidieR. :)
Good catch!
Looks like the failure is a snapshot update which is expected.
The warnings weren't appearing for me but seem to be related to
.data
for a few instances I used inclean_redcap_long()
. I think here:
I think I fixed this in the last commit. I'll keep an eye on the CI. I went ahead and just upgraded everything in our renv.lock
so package versions locally should be closer to what's running in CI.
Description
This PR fixes a bug that was causing one of our error messages to print incorrectly. Since it required learning a bit about custom print methods I also added a customization to the header when our supertibbles print. Let me know if you think we should keep it.
Proposed Changes
vec_ptype_abbr.redcap_supertbl()
method so that objects of the supertibble type get abbreviated tosuprtbl
in cli messages.vctrs
vignette which advises keeping labels to 6 characters because these labels also appear when objects are elements of atibble
orlist
tbl_sum.redcap_supertbl()
method so that printed supertibbles include a header with the object type and number of instrumentsutility/refresh.R
Error Message Fix
Old
New
Custom Header
Old
New
Issue Addressed
Closes #157
PR Checklist
Before submitting this PR, please check and verify below that the submission meets the below criteria:
.RDS
) updated underinst/testdata/create_test_data.R
usethis::use_version()
Code Review
This section to be used by the reviewer and developers during Code Review after PR submission
Code Review Checklist