[Buildbot] #2743: Create a hierarchy of review / merging
Buildbot trac
trac at buildbot.net
Sun Sep 7 16:50:31 UTC 2014
#2743: Create a hierarchy of review / merging
------------------------+-------------------------
Reporter: dustin | Owner: dustin
Type: enhancement | Status: assigned
Priority: minor | Milestone: botherders
Version: | Resolution:
Keywords: |
------------------------+-------------------------
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
> [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
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:
1. help other committers learn to merge pull requests
1. 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
1. two-stage merges, where pull requests go to another committer's repos
for testing, and I pull from that repo periodically
--
Comment (by sa2ajj):
BTW, option 3 sounds a bit too complicated.
--
Ticket URL: <http://trac.buildbot.net/ticket/2743#comment:3>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation
More information about the bugs
mailing list