Skip to content

Conversation

DavisVaughan
Copy link
Member

@DavisVaughan DavisVaughan commented Jan 6, 2023

Closes #1447
Closes #1446

  • Added error_call to the "backend" functions of pack(), unpack(), chop(), and unchop()
  • Added empty ... to unpack(), unchop(), and chop()
  • Disallowed renaming in all 6 of: nest(), unnest(), pack(), unpack(), chop(), and unchop()

"8c16af6a-b91e-47df-9707-5270b389ec14"

One change in revdeps from tidypaleo which was accidentally passing a named tidyselection through to unnest(). I'm happy to fix that one to get all 6 of these functions working consistently.

@DavisVaughan DavisVaughan force-pushed the feature/dots-and-error-calls branch from 9b65ffb to fe0ae9e Compare January 9, 2023 15:51
@@ -125,7 +125,6 @@ nest.data.frame <- function(.data, ..., .names_sep = NULL, .key = deprecated())

#' @export
nest.tbl_df <- function(.data, ..., .names_sep = NULL, .key = deprecated()) {
check_string(.names_sep, allow_null = TRUE)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Managed by pack() now

@DavisVaughan DavisVaughan requested a review from hadley January 9, 2023 21:16
@DavisVaughan DavisVaughan force-pushed the feature/dots-and-error-calls branch from 47fb604 to 52239e4 Compare January 10, 2023 15:27
@DavisVaughan DavisVaughan merged commit 1dd9113 into tidyverse:main Jan 10, 2023
@DavisVaughan DavisVaughan deleted the feature/dots-and-error-calls branch January 10, 2023 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add ... to unpack() and unchop() Add error_call to backend-ish functions
2 participants