kbendick opened a new pull request #4210:
URL: https://github.com/apache/iceberg/pull/4210


   Right now, we use both `push` and `pull_request` event to trigger our github 
actions.
   
   This causes us to duplicate our `paths-ignore` in every file.
   
   Additionally, as we start to use more github action features, such as in 
https://github.com/apache/iceberg/pull/4207, it would be good to reduce the 
number of events that trigger workflows so that the `github` context has 
consistent fields. For example, `github.head_ref` is only defined for 
`pull_request` and not `push` or any other trigger.
   
   Additionally, using `push` causes CI to run in forks etc when people push to 
their branch.
   
   Using the `pull_request` type of `synchronize` will make it so that CI is 
triggered when people update their PR, which was the original intention behind 
using `push` event.
   
   cc @snazy @nastra @rdblue 


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



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to