Skip to content

Commit ac62ddb

Browse files
committed
respond to review
1 parent 2bf591a commit ac62ddb

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

R/methods-epi_df.R

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -341,7 +341,7 @@ reclass <- function(x, metadata) {
341341

342342
#' Arrange an epi_df into a standard order
343343
#'
344-
#' Moves `key_colnames()` to the left, then arranges rows based on that
344+
#' Moves [key_colnames()] to the left, then arranges rows based on that
345345
#' ordering. This function is mainly for use in tests and so that
346346
#' other function output will be in predictable order, where necessary.
347347
#'
@@ -357,9 +357,8 @@ arrange_canonical <- function(x, ...) {
357357
#' @export
358358
arrange_canonical.default <- function(x, ...) {
359359
rlang::check_dots_empty()
360-
cli::cli_warn(c(
361-
"`arrange_canonical()` is only meaningful for an {.cls epi_df}.",
362-
i = "Returning the original {.cls {class(x)[1]}} object."
360+
cli::cli_abort(c(
361+
"`arrange_canonical()` is only meaningful for an {.cls epi_df}."
363362
))
364363
return(x)
365364
}

man/arrange_canonical.Rd

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)