Hello. On 09/09/2013 10:30 AM, Cedric BAIL wrote: > Hello, > > On Mon, Sep 9, 2013 at 5:37 PM, Stefan Schmidt <s.schm...@samsung.com> > wrote: >> On 09/09/2013 08:45 AM, Michael Blumenkrantz wrote: >>> I think I'd like to see patches being submitted/reviewed on the mailing >>> list at the least before giving blanket commit access because a > person's >> >> Our git setup allows commit rights to specific repos or even branches >> only. If we want we could limit it in a way people don't have full >> access to everything. >> >> On the other hand having a real pull model approach with maintainers >> reviewing the code, give feedback on it and pulling it in when ready >> would be nice. But to be honest I gave up on anything like this for the >> efl community. Yeah, its this can of worms again. :) > > We have phabricator and arcanist to start with here. Before giving > access to anyone, the first step is to see patch getting in with a > proper review. Then later they can get more. At least starting with > this step seems required for me.
One piece all you are missing here is that his commit access should not be for the efl projects anyway. He is supposed to work on a repo that is not even visible as it is marked as private. Its only the input things for wayland. I agree that we should not hand out commit access to easily but this case is just different. Its not related to the efl repos. regards Stefan Schmidt ------------------------------------------------------------------------------ Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more! Discover the easy way to master current and previous Microsoft technologies and advance your career. Get an incredible 1,500+ hours of step-by-step tutorial videos with LearnDevNow. Subscribe today and save! http://pubads.g.doubleclick.net/gampad/clk?id=58041391&iu=/4140/ostg.clktrk _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
