[Microbit-Python] Open sourcing the code

[Mark Shannon]

Hi Damien,

The way GitHub works is that in order to review and then merge code, the 
reviewer needs push rights.

As I said, I don't think anyone should merge their own code.
Ideally all code gets reviewed and merged by someone other than the 
author, after all we can all make mistakes.
But to do that, you need to give more people push access.

A bit of trust goes a long way.

Cheers,
Mark.


On 20/10/15 23:21, Damien George wrote:
> Hi Mark,
>
> We can't have too flat a hierarchy because then we'd lose control.
> Who/what do you mean by "most" in "most people should have push
> access"?
>
> The great thing about git(hub) is that anyone can clone the repo and
> hack on it themselves, without concent from the original
> author(s)/owner(s).  And then they can use such firmware on their
> microbit.
>
> Also, the real issue here is that the firmware generated from
> bbcmicrobit/micropython will be the firmware that kids use, and so it
> needs strict control as to what is commited.  That's going to be the
> controlling factor when it comes to push rights.
>
> Cheers,
> Damien.
>
>
>
>
> On Tue, Oct 20, 2015 at 7:34 PM, Mark Shannon <mark at hotpy.org> wrote:
>>
>>
>> On 20/10/15 08:46, Nicholas H.Tollervey wrote:
>>>
>>> Hi Folks,
>>>
>>> We're very please that the BBC have finally allowed us to open source
>>> our repos. As a result we're going to have to change the permissions on
>>> it now that it's vaguely public.
>>>
>>> The new repository is here:
>>>
>>> https://github.com/bbcmicrobit/micropython
>>>
>>> Its hosted under the auspices of the BBC because that gives us
>>> legitimacy and acknowledges we're actually part of the "official"
>>> micro:bit effort (which we've always been, but because of NDA have been
>>> unable to talk about).
>>>
>>> Damien has announced MicroPython's involvement in the project here:
>>>
>>> http://forum.micropython.org/viewtopic.php?f=8&t=1042
>>>
>>> I have blogged the historical context here:
>>>
>>> http://ntoll.org/article/story-micropython-on-microbit
>>>
>>> I'll be submitting something to tech news places later. Here's hoping
>>> people get interested. As a starter, here's the link to my submission on
>>> Hacker News:
>>>
>>> https://news.ycombinator.com/item?id=10417919
>>>
>>> Please up-vote..!
>>>
>>> Because we're now "open" we're going to use the standard GitHub
>>> fork->branch->PR methodology that people are used to.
>>
>>
>> Aren't we already doing that?
>>
>> Ergo, we're going
>>>
>>> to revoke push access for everyone except for Damien (and probably me as
>>> backup) - who will act as official gatekeeper.
>>
>>
>> Given that this is a nominally a BBC and/or PSF project (please correct me
>> if I'm wrong) wouldn't be better if we had a flatter hierarchy.
>> What I would suggest is that most people should have push access, but that
>> everyone (including Damien and yourself) should submit PRs which are then
>> merged by the reviewer.
>> That way all code gets at least two pairs of eyes on it before it is merged.
>>
>> I'm happy to review any PRs.
>>
>>>
>>> Please fork the repos!
>>>
>>> If this is a real pain in the neck for you we'll re-add you if you let
>>> us know.
>>>
>>> Best wishes,
>>>
>>> Nicholas.
>>>
>>
>> Cheers,
>> Mark.
>> _______________________________________________
>> Microbit mailing list
>> Microbit at python.org
>> https://mail.python.org/mailman/listinfo/microbit
> _______________________________________________
> Microbit mailing list
> Microbit at python.org
> https://mail.python.org/mailman/listinfo/microbit
>