[PATCH] D67262: [git-llvm] Do not reinvent `@{upstream}`

[Mehdi AMINI via Phabricator via llvm-commits]

mehdi_amini added a comment.

In D67262#1665677 <https://reviews.llvm.org/D67262#1665677>, @davezarzycki wrote:

> In D67262#1665661 <https://reviews.llvm.org/D67262#1665661>, @mehdi_amini wrote:
>
> > > I don't know what the long-term plans for this script are.
> >
> > The only way for us to enforce linear history on GitHub at the moment is to mandate the use of the script unfortunately.
>
>
> Food for thought:
>
> 1. git-llvm is run client side, so the linear history check isn't technically "enforced".


Someone would have to be really malicious to work around the server-side checks that goes with the client though. This is in the range of getting commit access removed IMO.

> 5. nag/beg GitHub to make merge commits optionally require project administrator privileges. I'd hope for a high-profile project like LLVM, they would consider it.

I think we requested it a couple of time already, I don't think it got anywhere though.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67262/new/

https://reviews.llvm.org/D67262