[Buildbot] #2743: Create a hierarchy of review / merging

Buildbot trac trac at buildbot.net
Sun Sep 7 13:56:15 UTC 2014


#2743: Create a hierarchy of review / merging
------------------------+-------------------------
Reporter:  dustin       |       Owner:
    Type:  enhancement  |      Status:  new
Priority:  minor        |   Milestone:  botherders
 Version:               |  Resolution:
Keywords:               |
------------------------+-------------------------
Description changed by sa2ajj:

Old description:

> When I get busy for 5-10 days at a time, Buildbot pull reqs pile up.
> This sucks for contributors, and when I *do* have time to work on
> Buildbot, it still takes me a few days to dig through the backlog.
>
> Options I can think of:
>
>  * help other committers learn to merge pull requests
>  * require contributors to get reviews done by someone else, against
> PatchGuidelines, so I only do a cursory check once that other review is
> complete
>  * two-stage merges, where pull requests go to another committer's repos
> for testing, and I pull from that repo periodically

New description:

 When I get busy for 5-10 days at a time, Buildbot pull reqs pile up.  This
 sucks for contributors, and when I *do* have time to work on Buildbot, it
 still takes me a few days to dig through the backlog.

 Options I can think of:

  * help other committers learn to merge pull requests
  * require contributors to get reviews done by someone else, against
 [wiki:PatchReview patch review guidelines], so I only do a cursory check
 once that other review is complete
  * two-stage merges, where pull requests go to another committer's repos
 for testing, and I pull from that repo periodically

--

--
Ticket URL: <http://trac.buildbot.net/ticket/2743#comment:1>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation


More information about the bugs mailing list