On 03/20/2019 11:38 AM, Zachary Turner wrote: > It sounds like we need to get someone from the Foundation (chandlerc@, > lattner@, tanya@, someone else?) to reach out to them offline about this. >
Yes, we will try to reach out to GitHub directly about this, but I still think we need some kind of contingency plan in case pre-receive hooks or even a new kind of branch protection won't be an option for us. -Tom > On Wed, Mar 20, 2019 at 11:23 AM Arthur O'Dwyer <arthur.j.odw...@gmail.com > <mailto:arthur.j.odw...@gmail.com>> wrote: > > On Wed, Mar 20, 2019 at 2:19 PM Tom Stellard via cfe-dev > <cfe-...@lists.llvm.org <mailto:cfe-...@lists.llvm.org>> wrote: > > On 03/20/2019 10:41 AM, Zachary Turner wrote: > > > > On Tue, Mar 19, 2019 at 12:00 PM Tom Stellard via lldb-dev > <lldb-dev@lists.llvm.org <mailto:lldb-dev@lists.llvm.org> > <mailto:lldb-dev@lists.llvm.org <mailto:lldb-dev@lists.llvm.org>>> wrote: > > > > Hi, > > > > I would like to follow up on the previous thread[1], where > there was a consensus > > to disallow merge commits in the llvm github repository, and > start a discussion > > about how we should enforce this policy. > > > > Unfortunately, GitHub does not provide a convenient way to > fully enforce this policy. > > > > > > Why isn't this enforceable with a server-side pre-receive hook? > > GitHub[1] only supports pre-receive hooks in the 'Enterprise Server' > plan, which is for self-hosted github instances. > > > AIUI, the GitHub team is perfectly willing to help out the LLVM project > in whatever way LLVM needs, including but not limited to turning on > server-side hooks for us. > https://twitter.com/natfriedman/status/1086470665832607746 > > Server-side hooks are *the *answer to this problem. There is no problem. > You just use a server-side hook. > > (Whether or not to use GitHub PRs is an orthogonal question. You can use > hooks with PRs, or hooks without PRs; PRs with hooks, or PRs without hooks.) > > –Arthur > _______________________________________________ lldb-dev mailing list lldb-dev@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev
