代码审查时审查员发现小瑕疵如何处理?

发布于 2022-09-02 00:22:57 字数 119 浏览 17 评论 0

团队通过github的pull request做代码评审,审查员发现小瑕疵比如js少了个分号、undefined值没做空串处理等,这种情况审查员自己就能改改,需要反馈给开发者自己修改吗?如果审查员改,如何处理别人的这个分支?

如果你对这篇内容有疑问,欢迎到本站社区发帖提问 参与讨论,获取更多帮助,或者扫码二维码加入 Web 技术交流群。

扫码二维码加入Web技术交流群

发布评论

需要 登录 才能够评论, 你可以免费 注册 一个本站的账号。

评论(1

似狗非友 2022-09-09 00:22:57

我的做法是自己改掉 通知当事人

如果同一当事人经常写这样的东西(不管是粗心还是不认为需要写), 另找时间教育

另外能用lint查出的小错误尽量做到CI里去

~没有更多了~
我们使用 Cookies 和其他技术来定制您的体验包括您的登录状态等。通过阅读我们的 隐私政策 了解更多相关信息。 单击 接受 或继续使用网站,即表示您同意使用 Cookies 和您的相关数据。
原文