diff --git a/NEWS.md b/NEWS.md index e2a0da8..5e0d5ba 100644 --- a/NEWS.md +++ b/NEWS.md @@ -1,8 +1,11 @@ # tidyselect (development version) +* Fix for CRAN checks. + * `eval_select()` out-of-bounds errors now use the verb "select" rather than "subset" in the error message for consistency with `dplyr::select()` (#271). + # tidyselect 1.2.0 ## New features diff --git a/R/lifecycle-deprecated.R b/R/lifecycle-deprecated.R index eb324a9..d8055ce 100644 --- a/R/lifecycle-deprecated.R +++ b/R/lifecycle-deprecated.R @@ -7,7 +7,7 @@ #' `vignette("tidyselect")` to get started. #' #' @param .vars A character vector of existing column names. -#' @param ...,args Selection inputs. See the help for [selection +#' @param ... Selection inputs. See the help for [selection #' helpers][language]. #' @param .include,.exclude Character vector of column names to always #' include/exclude. diff --git a/man/faq-external-vector.Rd b/man/faq-external-vector.Rd index 4631f22..789b48a 100644 --- a/man/faq-external-vector.Rd +++ b/man/faq-external-vector.Rd @@ -38,8 +38,7 @@ external object. \if{html}{\out{
}}\preformatted{vars <- c("cyl", "am", "vs") result <- mtcars \%>\% select(vars) -#> Warning: Using an external vector in selections was deprecated in tidyselect -#> 1.1.0. +#> Warning: Using an external vector in selections was deprecated in tidyselect 1.1.0. #> i Please use `all_of()` or `any_of()` instead. #> # Was: #> data \%>\% select(vars) @@ -47,11 +46,10 @@ result <- mtcars \%>\% select(vars) #> # Now: #> data \%>\% select(all_of(vars)) #> -#> See -#> . +#> See . #> This warning is displayed once every 8 hours. -#> Call `lifecycle::last_lifecycle_warnings()` to see where this -#> warning was generated. +#> Call `lifecycle::last_lifecycle_warnings()` to see where this warning was +#> generated. }\if{html}{\out{
}} We have decided to deprecate this particular approach to using external diff --git a/man/faq-selection-context.Rd b/man/faq-selection-context.Rd index d418979..6702747 100644 --- a/man/faq-selection-context.Rd +++ b/man/faq-selection-context.Rd @@ -13,23 +13,20 @@ Using a selection helper anywhere else results in an error: \if{html}{\out{
}}\preformatted{starts_with("foo") #> Error: #> ! `starts_with()` must be used within a *selecting* function. -#> i See -#> -#> for details. +#> i See for +#> details. mtcars[contains("foo")] #> Error: #> ! `contains()` must be used within a *selecting* function. -#> i See -#> -#> for details. +#> i See for +#> details. subset(mtcars, select = matches("foo")) #> Error: #> ! `matches()` must be used within a *selecting* function. -#> i See -#> -#> for details. +#> i See for +#> details. }\if{html}{\out{
}} If you see this error, you may have used a selection helper in the wrong diff --git a/man/vars_select.Rd b/man/vars_select.Rd index 43ac09d..891685a 100644 --- a/man/vars_select.Rd +++ b/man/vars_select.Rd @@ -18,7 +18,7 @@ vars_rename(.vars, ..., .strict = TRUE) \arguments{ \item{.vars}{A character vector of existing column names.} -\item{..., args}{Selection inputs. See the help for \link[=language]{selection helpers}.} +\item{...}{Selection inputs. See the help for \link[=language]{selection helpers}.} \item{.include, .exclude}{Character vector of column names to always include/exclude.}