[GitHub] [arrow] jonkeane commented on a diff in pull request #12980: ARROW-16281: [R] [CI] Bump versions with the release of 4.2

[GitBox <gi...@apache.org>]

jonkeane commented on code in PR #12980:
URL: https://github.com/apache/arrow/pull/12980#discussion_r870533018


##########
.github/workflows/r.yml:
##########
@@ -244,7 +244,7 @@ jobs:
         config:
         - { rtools: 35, rversion: "3.6" }
         - { rtools: 40, rversion: "4.1" }
-        # TODO: Once R 4.2 comes out we can switch to devel + 4.2
+        - { rtools: 42, rversion: "4.2" }

Review Comment:
   This is probably fine (and we should try it and see if it's an issue...) but this _does_ add an additional build to the CI that runs on PRs|commits, and we typically try and keep those as sparse as we can.
   
   But given the differences between roots + windows and our struggles maintaining them, adding a job for not is a good plan — but we might need to reassess if it adds too much waiting



-- 
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