Closed pat-s closed 4 years ago
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
R/name_chunks.R | 3 | 6 | 50.0% | ||
<!-- | Total: | 7 | 10 | 70.0% | --> |
Totals | |
---|---|
Change from base Build 75: | -2.4% |
Covered Lines: | 107 |
Relevant Lines: | 113 |
:wave: @pat-s! Thanks for your PR! We'll get to this PR soon-ish, but after the patch release we'll do to fix CRAN results. Thanks for your patience.
Once the patch release is on CRAN, I'll merge this manually since I created too many conflicts. :-) (local checks ok).
@pat-s thanks a lot! :rocket: And sorry for the delay! :snail:
Side note for future contributions, please open issues first so you're sure to not open an PR for nothing.
Thanks a lot @maelle!
Side note for future contributions, please open issues first so you're sure to not open an PR for nothing.
Will do!
name_chunks()
andname_dir_chunks()
are now able to unname all chunks before naming them. This ensures a consistent naming for all chunks instead of just labelling unnamed chunks.unname_dir_chunks()
that works in the same way asname_dir_chunks()
unname_all_chunks()
tounname_chunks()
to be consistend withname_chunks()
cli
verbosityunname
forname_chunks()
andname_dir_chunks()
unname
NEWS.md needs to be updated with the PR number before merge.