javascript - 代码审查时审查员发现小瑕疵如何处理?
迷茫
迷茫 2017-04-10 16:48:57
[JavaScript讨论组]

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

迷茫
迷茫

业精于勤,荒于嬉;行成于思,毁于随。

全部回复(1)
高洛峰

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

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

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

热门教程
更多>
最新下载
更多>
网站特效
网站源码
网站素材
前端模板
关于我们 免责申明 意见反馈 讲师合作 广告合作 最新更新 English
php中文网:公益在线php培训,帮助PHP学习者快速成长!
关注服务号 技术交流群
PHP中文网订阅号
每天精选资源文章推送
PHP中文网APP
随时随地碎片化学习
PHP中文网抖音号
发现有趣的

Copyright 2014-2025 https://www.php.cn/ All Rights Reserved | php.cn | 湘ICP备2023035733号