[wip] Shorten help for rows
and columns
arguments.
#1669
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Not yet ready for review !
Summary
The first goal was to remove the mention the tidyselect::one_of (superseded by any_of + all_of.
Than, I realized there was a bit of duplication, so I looked into that.
everything()
, shorten how the default is explained + don't link toeverything()
twice in manc()
too often.rows
andcolumns
is explained (since that knowledge applies to all of gt, and not to any particular functions.@inheritParams
safely without changing anything in man. than reopen a PR with the same intention as this one, but with minimal changesI am opening to check-out my progress.
I will review shortly to ensure consitency, and will use
@inheritParams
as much as possible, unless some special info is shared.i.e. in
cols_hide()
, I think it is worth mentioning how to hide columns with all missing values. 8901a1fBefore requesting review, I want to double-check that the style used is consistent.
I used r-lib/roxygen2#1594 to document, for faster iteration.