[GitHub] [iceberg] singhpk234 commented on pull request #4989: Infra - Fix API compatibility Github Action

[GitBox <gi...@apache.org>]

singhpk234 commented on PR #4989:
URL: https://github.com/apache/iceberg/pull/4989#issuecomment-1149497348

   Thanks @kbendick for the change and comprehensive deep-dive, the above approach LGTM !!
   
   [question] : based on this [article](https://frontside.com/blog/2020-05-26-github-actions-pull_request/#how-does-pull_request-affect-actionscheckout)
   
   > that a pull_request workflow `ref` would look like refs/remotes/pull/##/merge, SHA of a pull_request workflow doesn't  matches the commit that triggered the workflow; instead the SHA of the pull_request is the resulting commit that was created from merging the base to the head.
   
   Are we ok failing in the CI incase there is a merge conflict when merging base to the head ? Thoughts.
    (I think we should be ok, considering we need to resolve them any how before final merge.)
   


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org