[bind10-dev] Code review procedure

Mukund Sivaraman muks at isc.org
Wed Jan 29 13:13:24 UTC 2014


On Wed, Jan 29, 2014 at 06:41:43PM +0530, Mukund Sivaraman wrote:
> Please be aware of the BIND 10 code review procedure:
> http://bind10.isc.org/wiki/CodeReviewProcedure
> 
> A reviewer should make sure that the branch builds on their machine,
> passes unit tests and lettuce tests (lettuce may be optional for DHCP),
> is exception safe and has exception documentation, API documentation,
> that "distcheck" target passes and .gitignore has been updated (if
> necessary) when any files have been added or removed, etc.

BTW, all this applies just as well to the developer putting a branch to
review too. :) The reviewer just double-checks all this.

		Mukund
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 2881 bytes
Desc: not available
URL: <https://lists.isc.org/pipermail/bind10-dev/attachments/20140129/f5fcaaf8/attachment.bin>


More information about the bind10-dev mailing list