jonkeane commented on PR #12867: URL: https://github.com/apache/arrow/pull/12867#issuecomment-1098080576
Awesome, I've kicked off the CI again (sorry about the delay on that! Feel free to @ me when you need that approved again — it'll only be needed for this first PR!) I know this wasn't in the original ticket, but would you mind also removing the lines that set the testthat edition: `local_edition(2)` and `local_edition(3)` https://github.com/apache/arrow/blob/18803eaafdea1994da42c5ea4bd8cf046057c357/r/tests/testthat/test-dplyr-summarize.R#L300-L302 https://github.com/apache/arrow/blob/18803eaafdea1994da42c5ea4bd8cf046057c357/r/tests/testthat/test-dplyr-summarize.R#L345 (and then there are at least two more later on...) and then add `SuppressWarnings()` around the tests that have multiple warnings? -- 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]
