raulcd commented on PR #13188: URL: https://github.com/apache/arrow/pull/13188#issuecomment-1131556113
Hi @kszucs, I think this is ready for review, it has been slightly tricky to validate so I am going to explain on the comment here. There are two commits on this PR, one with the change of using the 10 digit hex as part of the action branch name and one to modify the comment bot in order to pick the PR changes on archery instead of always from master. This second commit will allow us to be able to test changes for the comment bot from the PR. I have tested on my [own fork ](https://github.com/raulcd/arrow) by: * Merging to my master branch [the commit about the comment bot](https://github.com/raulcd/arrow/commit/0ab58a57708484ab40ca524cf62a2159e2ff3954) so it is able to checkout the PR instead of master as seen on [the comment bot action triggered](https://github.com/raulcd/arrow/runs/6505894312?check_suite_focus=true) * Creating a [PR with the changes on the crossbow submit action](https://github.com/raulcd/arrow/pull/2) on my own repo and validate the [branch created on crossbow](https://github.com/ursacomputing/crossbow/actions?query=branch:actions-364c9d4277-github-example-cpp-minimal-build-static) is the correct one Please let me know your thoughts and if you would like me to split the two commits on two different issues as they are solving slightly different issues. -- 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]
