代码审查时审查员发现小瑕疵如何处理?
团队通过github的pull request做代码评审,审查员发现小瑕疵比如js少了个分号、undefined值没做空串处理等,这种情况审查员自己就能改改,需要反馈给开发者自己修改吗?如果审查员改,如何处理别人的这个分支?
如果你对这篇内容有疑问,欢迎到本站社区发帖提问 参与讨论,获取更多帮助,或者扫码二维码加入 Web 技术交流群。
绑定邮箱获取回复消息
由于您还没有绑定你的真实邮箱,如果其他用户或者作者回复了您的评论,将不能在第一时间通知您!
团队通过github的pull request做代码评审,审查员发现小瑕疵比如js少了个分号、undefined值没做空串处理等,这种情况审查员自己就能改改,需要反馈给开发者自己修改吗?如果审查员改,如何处理别人的这个分支?
由于您还没有绑定你的真实邮箱,如果其他用户或者作者回复了您的评论,将不能在第一时间通知您!
接受
或继续使用网站,即表示您同意使用 Cookies 和您的相关数据。
发布评论
评论(1)
我的做法是自己改掉 通知当事人
如果同一当事人经常写这样的东西(不管是粗心还是不认为需要写), 另找时间教育
另外能用lint查出的小错误尽量做到CI里去