Re: Github Status Checks


Tim Meehan
 

Hello,

 

As we have all related company’s voting members of the TSC submitting their vote in favor, voting is now concluded and we’ll enable the status checks shortly.  Thank you!

 

Tim

 

From: Tim Meehan <tdm@...>
Date: Tuesday, May 4, 2021 at 8:49 PM
To: presto-tsc@... <presto-tsc@...>
Subject: 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

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