[关闭]
@universal 2019-09-12T06:37:00.000000Z 字数 610 阅读 397

Emergencies (紧急事件)

codeReview


什么是Emergency ChangeList

一个紧急的变更:并不是指需要代码回滚的情况。指的是在正常代码开发流程中提交的小型的 MR,但是优先级较高,比如解决影响正式环境用户的bug,修复一些安全漏洞。

进行codeReview的人此时关注的要点是做CR的时间以及代码的正确性(即是否解决了问题)。
等问题解决后还需要再对提交的代码进行详细的CR。(按照代码审查规范来)

哪些不是Emergency ChangeList

  1. 比如某些功能需要提前一周发布,而不是原定的时间发布
  2. 开发者已经开发了很长时间,想要快速通过MR提测
  3. reviewers不方便的情况,比如不在办公区域或者工作时间
  4. 在工作日的最后一天提交的MR
  5. 代码回滚导致的测试或者构建失败,然后做的改正提交。
  6. 这个需求/提交的soft deadline是今天

什么是Hard DeadLine

hard dealine是指如果错过这个最后期限,会导致一系列灾难性后果的发生。
比如约定好的合同时间限制,或者产品错过当前时间发布就会失去市场地位。
注意:延迟一周发布并不是灾难性的。
很多的deadlines都是soft的,表示希望在规定时间内完成某项功能,但是不能以牺牲代码健康为代价。
最后如果在开发周期内有较大的提交的话,尽量提前提交,避免在开发周期的最后一天提交,以保证有足够的时间进行CR。开发人员也要合理分配每次MR的代码量,不求多但求稳。

添加新批注
在作者公开此批注前,只有你和作者可见。
回复批注