Re: Allowing external checks on Github PR


Ariel Weisberg
 

Non-binding +1

This is great!

On Wed, Mar 25, 2020, at 6:06 PM, Leiqing Cai wrote:
I'd like to propose allowing additional external checks on Github PR. Specifically, I'm looking to a test to check for Facebook integration.

Currently, Travis CI is run on each PR (shown in the screenshot below).
However, this is not enough since there are active repos depending upon the master of prestodb/presto.
Allowing adding external integration checks can help identify breakages earlier on.

Here are the rules that I propose for external checks:
- Anyone can add external checks upon TSC approval.
- Test owner should make sure that the test is not flaky.
- Test failures does not block merging. They only serves alerting purpose.



Attachments:
  • Screen Shot 2020-03-25 at 2.41.18 PM.png

Join presto-dev@lists.prestodb.io to automatically receive all group messages.