[GitHub] [iceberg] kbendick commented on pull request #4210: Infra - Trigger CI using on pull_request instead of on push

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

kbendick commented on pull request #4210:
URL: https://github.com/apache/iceberg/pull/4210#issuecomment-1049103854


   The one thing I think this might affect is that we currently implicitly rely on the `push` event for running CI when merging into master.
   
   This has the unfortunate side effect of cancelling merges into master if another merge happens (because of the cancel duplicates work). Ideally, all CI would run to completion after merging into master, so commits don't look like they failed CI.
   
   I will also open a PR to fix that as well.


-- 
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