由Jesal Gadhia
拉请求(PRs)是一个伟大的方式提交对项目的贡献,尤其是当有多个开发人员在同一时间。如果做得好,他们是一个媒介,通过它我们可以接收反馈,提高能见度的变化我们航运。
有许多元素是什么让一个很好的公关。他们最终确定最终可交付成果的质量以及讲故事的某些特性或变化是怎么来的。
在这篇文章中,我将介绍一些基本的优化什么公关在每个阶段的生产之旅。我希望你可以使用这个反馈质量和最小化最大化时间合并。所以就让我们一探究竟吧。
范围
制定公关之旅的第一步是创建一个单独的分支为你改变。在这个阶段,范围变更是关键。首先,认为多大的一个用于你想介绍一个公关。试图解决一个问题或构建一个功能在公关。更大的,更复杂的是审查和更有可能会被推迟。记住,回顾PRs是花时间从别人的一天!
一旦你确定了范围,确保每个变化分解成小的逻辑提交。有小提交有价值是因为以下几个原因。首先,它允许审查者看到的发展变化和理解公关在其当前状态。如果您正在使用也是非常有价值的git平分调试一个问题或试图遏制未来的变化。
起草
旅程的下一步是打开一个公关草案。这并不总是必要的,但它是有用的,如果你想看到完整的CI运行所有的测试,如果你不是在本地运行整个测试套件。这会给你您的更改是否早期信号与其余的代码库以意想不到的方式相互作用。
花一些时间来回顾你的变化,看看是否一切从审稿人的角度看是有道理的。如果你公关的状态感到满意,你可以标记评论家在必要时早期反馈。
如果你决定标签评论者,要求有具体的反馈,呼唤的公关还是半成品。指导你的评论家与你想要的反馈。这将确保他们不花时间和精力给你反馈的事情可能会改变,或者你已经知道的事情。
出版
如果你的草案公关情况良好,现在可以标记评论家。当谈到选择审稿人,挑选不超过2 - 3主评审员。如果你标记一个大量的评论员,那么你可能会看到一个旁观者效应。
标签评论者之前,确保:
- 测试都是绿色的。你不希望评论者花宝贵的时间指出基础。
- 你已经清楚地说明了公关的目的以及变化的大背景下。相关链接回到松弛线程或门票。这是很重要的,即使你认为评审人员将完整的上下文为了子孙后代。
- 你做了一个自我回顾。我建议通过运行马斯洛的金字塔的代码评审如果你的工作符合这些标准。你应该第一个批判性回顾你的工作在别人之前。
- 如果有你心目中的具体跟进,描述那些所以评论家知道会发生什么。
- 你已经提供了相关的工件公关以截图的形式,视频或日志。这将帮助您的审阅人员更好地理解您的更改的影响。
反馈
当你收到反馈评审人员请记住:
- 语气很容易迷失在异步通信。所以认为最好的意图,不要把个人的反馈。为你的决定提供澄清和上下文。
- 如果有什么不清楚,请澄清问题,不要试图阅读茶叶。如果事情还不清楚在异步谈话,要求一些时间在打电话聊天。一旦你有机会讨论,回头总结讨论分公关保护环境。
- 你是否决定接受反馈,一定要确认你已经收到的反馈。这说明你重视评审人员的时间和精力。
- 明确re-request审查一旦你完成解决反馈。不要认为审稿人就会知道当你完成。加分如果后续评论指出解决的具体提交他们的反馈。
释放
一旦你已经通过了反馈周期和得到批准,评论者记得回来和合并你的公关,你等待的时间越长,遇到合并冲突的机会就越大或回归。所以一定要尽快合并。
如果你是释放一个公关执行大量数据操作或改变一些关键路径,确保您可以按照公关通过发布过程和减轻任何错误或打嗝。
如果一切顺利,你的变化是在生产中,还记得与利益相关者和后续更新它们的进展。
关于作者
Jesal热衷于应用技术对有意义的,积极的影响。他在中学开始写代码为了好玩。从那时起,他积累了超过十年的经验在科技行业从一个工程师的创始人之一。在空闲时间,他喜欢玩他的儿子,修修补补的项目,旅行。
加入谈话