介绍评论轮盘
在Moxio,我们最近开始了一个名为“审阅轮盘”的实验:随机代码审阅的过程。 我们认为这强调了代码审查作为双向学习的一种手段,并有助于新手入门,因此对我们先前的审查“政策”有所改进。 在这篇文章中,我想勾勒出该实验背后的背景,解释“ Review Roulette”的想法,并提出一些初步结果。 我们的旧情况 在引入Review Roulette之前,我们已经进行了大量代码审查。 大约四年前,我们在本地网络上建立了一个ReviewBoard服务器,并开始使用它来讨论和审查代码。 审核将在该工具中进行,如果需要,可以使讨论脱机。 到目前为止,我们还没有关于什么以及如何进行审查的正式政策:可以选择进行变更进行审查,并且作者选择了审查者和提交时间(提交前或提交后)。 有时,复审请求只是一个粗略的讨论想法; 在其他时候,将有完整的实施方案来验证。 这种方法使开始进行代码审查变得容易,并确保其引入或多或少没有阻力。 它还使审查负担和官僚管理负担保持在较低水平,从而轻松进行小规模修复和改进(本着童子军规则的精神),而无需经过正式的审查流程。 然而,这种松散的结构也具有其缺点。 在实践中,选择加入代码审查意味着大多数要审查的主题都是较困难或“有争议的”变更,这些变更势必引发大量的讨论(他们经常这样做)。 这给代码审查带来了太多麻烦的讨论和艰难的决定。…