<div dir="ltr"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 9, 2019 at 12:39 AM David Chisnall via cfe-dev <<a href="mailto:cfe-dev@lists.llvm.org">cfe-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">On 09/04/2019 00:32, Tom Stellard via cfe-dev wrote:<br>
> As a backup plan, I am going to look into implementing option #1, but instead<br>
> of having git-llvm create a pull request, it would push first to a staging<br>
> branch and then push to master if a 'no-merge commit' status checks pass.  This<br>
> is essentially the same flow as using pull requests, but without all the<br>
> pull request noise.<br>
<br>
I don't think that you can enforce this policy on the server side.  You <br>
could; however, make git-llvm create a pull request and, if the relevant <br>
checks pass, automatically use the GitHub APIs to do a rebase and merge.<br></blockquote><div><br></div><div>I believe you can with: <a href="https://help.github.com/en/articles/enabling-required-status-checks">https://help.github.com/en/articles/enabling-required-status-checks</a></div><div><br></div><div>I don't even think you need to push a branch for this*, the git-llvm script should be able to perform the required check locally, then issue a REST call and push to master.<br></div><div><br></div><div>-Mehdi</div><div><br></div><div>* on another note, I hope we will *disallow* user branches on the main repo in favor of forks.</div><div><br></div></div></div></div>