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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
