You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/04/13 13:55:03 UTC

[GitHub] [arrow] jonkeane commented on pull request #12867: ARROW-14168: [R] Warn only once about arrow function differences

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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org