[lldb-dev] [cfe-dev] [GitHub] RFC: Enforcing no merge commit policy
Zachary Turner via lldb-dev
lldb-dev at lists.llvm.org
Wed Mar 20 11:38:03 PDT 2019
It sounds like we need to get someone from the Foundation (chandlerc@,
lattner@, tanya@, someone else?) to reach out to them offline about this.
On Wed, Mar 20, 2019 at 11:23 AM Arthur O'Dwyer <arthur.j.odwyer at gmail.com>
> On Wed, Mar 20, 2019 at 2:19 PM Tom Stellard via cfe-dev <
> cfe-dev at 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 at lists.llvm.org <mailto:lldb-dev at lists.llvm.org>> wrote:
>> > Hi,
>> > I would like to follow up on the previous thread, 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 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.
> 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.)
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the lldb-dev