查看: 1230| 回复: 6
跳转到指定楼层
上一主题 下一主题
收起左侧

【求助】回复都加米!带的同事老是写Bug怎么办

全局:

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看附件。没有帐号?注册账号

x
组里有个小老弟做同一部分业务,但是这个小老弟的代码质量堪忧,平时大部分时间是我给他review PR,经常会有corner case想不到,而且如果不是我review的代码,我偶尔去看的之后经常会发现bug。但是data pipeline这个东西,很多时候是没法写test case的,所以对该怎么让他提高代码质量没什么头绪。而且officially我也不是他的mentor,不知道该怎么给他feedback。

想请教下大家,这种情况下1) 我该怎么告诉他代码质量有问题,直接说还是反馈给manager? 2) data pipeline有没有啥好的方法帮助保证代码质量?

上一篇:南加 Amazon
下一篇:借车给朋友开,要不要负责任
全局:
直接跟他说喽。不行的话就让别人来review。 至于测试 总有办法拆分吧。 冒烟测试总要搞搞吧

评分

参与人数 1大米 +1 收起 理由
胡椒牛柳 + 1 赞一个

查看全部评分

回复

使用道具 举报

🔗
longpp 2022-8-11 03:49:49 来自APP | 只看该作者
全局:
Manual test,如果没有那是你们公司的问题,换公司吧

评分

参与人数 1大米 +1 收起 理由
胡椒牛柳 + 1 赞一个

查看全部评分

回复

使用道具 举报

🔗
 楼主| 胡椒牛柳 2022-8-12 16:21:55 来自APP | 只看该作者
全局:
longpp 发表于 2022-08-10 12:49:49
Manual test,如果没有那是你们公司的问题,换公司吧
good idea🤔code review的时候可以问问都做了哪些测试
回复

使用道具 举报

🔗
zcdlwds 2022-8-15 11:20:47 | 只看该作者
全局:
我其实建议如果不会发生什么critical impact的bug你可以适当对一两个bug睁一只眼闭一只眼,等到出问题的时候让小老弟感受一下修这些Issue的痛苦。我觉得一般被现实这么教育一两次之后都会学乖😜
回复

使用道具 举报

🔗
mungo 2022-8-15 14:02:42 来自APP | 只看该作者
全局:
教教他如何写单元测试,然后沟通一下他的想法,大多数情况下都是不成熟的想法导致的(比如bug应该由测试的人负责)
回复

使用道具 举报

🔗
qf1224 2022-8-29 23:44:43 | 只看该作者
全局:
code review的时候每次都问他有没有做测试,都做了什么测试。或者建议他在PR summary里加上 test plan
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号

本版积分规则

>
快速回复 返回顶部 返回列表