jgiannuzzi commented on code in PR #47811: URL: https://github.com/apache/arrow/pull/47811#discussion_r2445458389
########## .github/workflows/cpp.yml: ########## Review Comment: Happy to do it! Before I do, I'd like to explain the reason I changed this workflow in the first place: I was trying to keep the 2 Windows msvc workflows identical so that they can ultimately be merged. That's why I have a 1-value matrix in cpp_extra.yml and that's why I put a comment on top of each workflow saying they should be kept aligned. If none of that resonates with you, I'm happy to revert the changes to cpp.yml and simplify the new workflow. -- 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]
