nealrichardson commented on PR #13441: URL: https://github.com/apache/arrow/pull/13441#issuecomment-1168109321
> 👎 In regards to adding `r-binary-packages` to the r-group. I think we should take a closer look at the PR CI `r.yml` and see if we want to change it to be more in-line with `r-binary-packages` as they are quite similar instead of adding the task to the r-group. The heavy lifting happens in both jobs already (windows-cpp) and I think `r.yml` should have a macos build anyway. I don't understand the objection. When I do a PR that affects different build settings in R, I want to run `crossbow submit -g r` and know that I've tested everything. To me, it doesn't make sense for that not to be exhaustive, even if it is somewhat wasteful. Yes, there is some redundancy, but this workflow does different work--clearly, otherwise I would have caught this in ARROW-16510, where I checked (what I thought was) all of the R workflows before merging. -- 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]
