[Buildbot-devel] Changes to review process

Dustin J. Mitchell dustin at buildbot.net
Sun Sep 7 19:23:41 UTC 2014


http://trac.buildbot.net/ticket/2743

We're switching up the review process a little bit, so that reviews
and merges are done by different people.

The new process is at http://trac.buildbot.net/wiki/PatchReview

The practical upshot is that we'll need more people watching for new
pull requests and performing review on them.  The committers on the
project will then swoop in to perform merges on pull requests labeled
"merge-me".

If you've been around the project for a while, please consider
subscribing to the buildbot-commits list [1] and reviewing patches as
they come in.  Partial reviews ("this part looks good but this other
part is over my head") are great, too.

If you're new to the project, and your patch is languishing, please
feel free to make some noise on the pull request, the mailing list, in
irc, or just by yelling out your window.  If we're not reviewing in a
timely fashion, that's a problem we should know about.

Thanks!

Dustin

[1] https://lists.sourceforge.net/lists/listinfo/buildbot-commits




More information about the devel mailing list