[Users] Gerrit

[Roland Haas]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hello all,

> Gerrit manages the process of code reviewing in git, and has a lot
> of customization. You can set things up so that no one can push to
> the main branch unless their change has been reviewed. You can make
> rules like, one senior developer and two junior developers must
> review, etc. By helping to enforce code reviews, Gerrit helps to
> increase code quality, and helps to train junior developers in
> understanding the codebase. I think it might be a good alternative
> to Trac.
You mean replacing Trac with Gerrit? Personally I would try to avoid
changing the user-facing interface of our bug tracking system. There
seem to be some review systems that plug into Trac eg
http://trac-hacks.org/wiki/PeerReviewPlugin that try to formalize the
review process. Our current method ie. setting the "review" state and
waiting for a "Please apply" would be replaced by some more formalized
method.

If it is possible to run Gerrit as a review system only (and somehow
hook it up into Trac) then I'd find that a better approach.

I do not think that restricting commits to only reviewed commits is a
good idea (others may very well disagree). At least not for projects
that have an active and (essentially) single author like Carpet or Kranc.

If there is anything I do not like about the current procedure it is
not actually the way that reviews are handled but more the issue that
sometimes review requests sit idle for a long time.

Similarly if the goal is to reduce the number of open tickets, we
could adopt SpEC's practice of having explicit ticket review code
calls where each ticket is discussed in a 5min slot (enforced) and at
the very least assigned to a person.

Yours,
Roland

- -- 
My email is as private as my paper mail. I therefore support encrypting
and signing email messages. Get my PGP key from http://keys.gnupg.net.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.14 (GNU/Linux)
Comment: Using GnuPG with Icedove - http://www.enigmail.net/

iEYEARECAAYFAlI3YBEACgkQTiFSTN7SboWueQCcCA3g4t5OJdjFonDnBjU7NTOq
X1MAoIDiNy7R0tIgWYHsIFDja8KOvG4S
=lVTG
-----END PGP SIGNATURE-----