Closed rainyq69 closed 2 years ago
还没有 ready 的 PR,可以设置为 draft 模式。这样不会启动 CI,也是提醒 reviewer 不要过早介入 review
还没有 ready 的 PR,可以设置为 draft 模式。这样不会启动 CI,也是提醒 reviewer 不要过早介入 review
收到,以后会注意这个细节
这次 commit 的改动量太大了,很不方便 review,下次一定注意。
还是需要在第一个 comment 里简要描述这个 PR 做了什么。如果因为内容多,分散到了多个 comment 里,那也需要做好索引和超链接。
否则其他人来 reviewer 上手没有侧重点,时间会耗费比较多。
还是需要在第一个 comment 里简要描述这个 PR 做了什么。如果因为内容多,分散到了多个 comment 里,那也需要做好索引和超链接。
否则其他人来 reviewer 上手没有侧重点,时间会耗费比较多。
好的,我把描述放在第一个 comment
还没有 ready 的 PR,可以设置为 draft 模式。这样不会启动 CI,也是提醒 reviewer 不要过早介入 review
收到,以后会注意这个细节
我才意识到,这个 PR 还没有 ready,所以我不用开始 review。不过 PR 是可以随时在 “正式” 和 draft 间转换的。
我先给你转程 draft 了。不过这个不影响你和其它同学讨论,互相提前 review
第 1 阶段
第 2 阶段
第 3 阶段
第 4 阶段
第 5 阶段
第 6 阶段
需要 review 的内容有:
欢迎 review !