Skip to content

Commit

Permalink
Merge pull request #5 from ybc82/patch-1
Browse files Browse the repository at this point in the history
Update emergencies.md
  • Loading branch information
rootsongjc authored Oct 26, 2020
2 parents 928a9a7 + d9a4bba commit b1d7be1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions content/docs/review/emergencies.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ title: 紧急情况

## 什么是紧急情况? {#what}

紧急 CL 是这样的****更新:允许主要发布继续而不是回滚,修复显着影响用户生产的错误,处理紧迫的法律问题,关闭主要安全漏洞等。
紧急 CL 是这样的****更新:允许主要发布继续而不是回滚,修复显著影响用户生产的错误,处理紧迫的法律问题,关闭主要安全漏洞等。

在紧急情况下,我们确实关心 Code Review 的整体速度,而不仅仅是响应的速度。仅在这种情况下,审查人员应该更关心审查的速度和代码的正确性(是否解决了紧急情况?)。此外(显然)这类状况的审查应该优先于所有其他 code reivew。

Expand Down Expand Up @@ -39,4 +39,4 @@ title: 紧急情况

大多数截止日期都是软截止日期,而非最后期限。软截止日期表示希望在特定时间内完成某项功能。它们很重要,但你不应该以牺牲代码健康为前提来达到。

如果您的发布周期很长(几周),那么在下一个周期之前就可能会牺牲代码审查质量来获取功能。然而,如果重复这种模式,往往会给项目建立压倒性技术债务。如果开发人员在周期结束时经常提交 CL,只需要进行表面评审就必须“进入”,那么团队应该修改其流程,以便在周期的早期发生大的功能变更,并有足够的时间进行良好的审查。
如果您的发布周期很长(几周),那么在下一个周期之前就可能会牺牲代码审查质量来获取功能。然而,如果重复这种模式,往往会给项目建立压倒性技术债务。如果开发人员在周期结束时经常提交 CL,只需要进行表面评审就必须“进入”,那么团队应该修改其流程,以便在周期的早期发生大的功能变更,并有足够的时间进行良好的审查。

0 comments on commit b1d7be1

Please sign in to comment.