paleolimbot commented on PR #13786: URL: https://github.com/apache/arrow/pull/13786#issuecomment-1231639562
I think you nailed it - the `mutate()` tests are about matching dplyr's functionality (e.g., they will fail if dplyr changes something and makes this code out of sync); the `expand_across()` tests are about `expand_across()`; and we need both. I would personally just copy/paste all the places where `across()` appears in your mutate tests but `expect_identical(expand_across(quo(across(...)), ...)` instead. If you feel this is out of scope for this PR I am OK with that, too (but I think it's worth trying). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
