On Thu, Sep 30, 2010 at 08:31, Daniel Stutzbach <[email protected]> wrote: > On Thu, Sep 30, 2010 at 9:52 AM, <[email protected]> wrote: >> >> Of course, this is only true if the core developers *do* submit to the >> same rules. Is anyone proposing that current core committers have all their >> work reviewed before it is accepted? > > I think most would welcome (or at least tolerate ;) ) additional review of > their code. > The hard part is encouraging contributors to find the time and motivation to > thoroughly review code that they aren't personally interested in (and > perhaps not even familiar with).
Once we have a good workflow in place we would have to start shifting our development culture towards requiring a review of code no matter who the author is (which I support doing). As for needing some point system or incentive to drive people to do it, I don't think that would be necessary. We are lucky to have a dev team that is friendly and respectful. I suspect that e.g., if Nick started doing reviews of importlib patches I would reciprocate the favour by reviewing runpy patches. Those of us who don't do reviews would most likely then just end up with patches sitting in the tracker waiting for a review. _______________________________________________ Python-Dev mailing list [email protected] http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com
