nealrichardson commented on PR #13441: URL: https://github.com/apache/arrow/pull/13441#issuecomment-1168645059
> @nealrichardson Maybe I phrased this to strongly. It's not really an objection and should not block this PR. My intention was to start a discussion about/review of the `r.yml` CI and if it does what we want&need or if we should extend it. You are right, it does make sense that all R related jobs are in the R group to prevent issues like this happening. 👍 cool, we can review it. We've selected builds in the r.yml in order to catch the widest spectrum of issues with the fewest builds, but it's always worth evaluating if that's still the case. Currently, that's (1) separate C++ shared library build with everything on, force_tests: true; (2) bundled static build on centos-7/gcc 4.8; (3) windows, all rtools versions (we once pushed the rtools35 build to nightly but it kept failing because people would merge things that passed CI but broke the build, so we moved it back). -- 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]
