+1

 

From: <presto-tsc@...> on behalf of "Tim Meehan via lists.prestodb.io" <tdm=fb.com@...>
Reply-To: Tim Meehan <tdm@...>
Date: Tuesday, May 4, 2021 at 8:49 PM
To: "presto-tsc@..." <presto-tsc@...>
Subject: [presto-tsc] Github Status Checks

 

Hello,

 

As mentioned in the TSC meeting today, I’d like to enable the Github status checks requirement to the master branch, which means status checks must pass before a PR can be merged.  This is to prevent accidental merge of a failing test into the master branch, or to prevent known failing tests from affecting other PRs.  If there’s unrelated flaky tests, then the idea is we should fix these tests or mark them as disabled.

 

Documentation on Github status checks are here: https://docs.github.com/en/github/administering-a-repository/about-protected-branches#require-status-checks-before-merging

 

Please give this a +1 if you agree that code should not be merged with known test failures.

 

Thanks

Tim