最近招了两个开发岗的下属,写完代码一点不测试,就开始push代码

H
Helen_Wood
楼主 (北美华人网)
招了两个人,其中一个人,代码写完了,从来不做unit test, 文档中的5个需求,做完3个,就开始提交代码,然后开始刷手机,
我还得帮他测试,发现需求少了2点没做,发邮件,然后他做完一个需求,又开始push代码,
我把代码pull下来一跑,10个错误,然后找他,他立即改完,又push上去,我又得pull下来,一跑,还有5个错误没改。
最后发现还有一个需求没做,找到他,他就开始question 这个需求不合理,用户根本不会用到。
招了他简直,累死了, 但凡他写完了,自己花30秒测试一下,都会省去其他人半个小时的时间。
另外一个下属就非常好,写完代码,自己做好unit test,逐项检查开发文档,确认无误才push代码,从来没任何问题。
R
Rachel1205
书面提出你的要下属完成的任务,截止日期,完成的标准。然后他们还做不到,你就开始走流程赶人吧。挤牙膏式的员工是态度问题,不好带。 千万要好声好气,不要失去耐心。
无欲则刚
不测试就push代码,听着像是微软的人。
一个用户
第一位大侠是天竺高手吗?
我猜他会说他上家公司有个QA team。
m
mt.everest
你需要吧delta coverage纳入cicd 另外就是每个feature都要有对应subtask
h
happymc
真累啊
王力宏
你们的ci/cd有问题。code test coverage 不够,怎么可以push。都走不到code review这一步。
r
realsimplejill
Helen_Wood 发表于 2024-10-10 05:35
招了两个人,其中一个人,代码写完了,从来不做unit test, 文档中的5个需求,做完3个,就开始提交代码,然后开始刷手机,
我还得帮他测试,发现需求少了2点没做,发邮件,然后他做完一个需求,又开始push代码,
我把代码pull下来一跑,10个错误,然后找他,他立即改完,又push上去,我又得pull下来,一跑,还有5个错误没改。
最后发现还有一个需求没做,找到他,他就开始question 这个需求不合理,用户根本不会用到。
招了他简直,累死了, 但凡他写完了,自己花30秒测试一下,都会省去其他人半个小时的时间。
另外一个下属就非常好,写完代码,自己做好unit test,逐项检查开发文档,确认无误才push代码,从来没任何问题。

纯好奇不测试就push code的人是怎么通过招聘流程的
j
jarvi
你怎么招到不合格的开发人的呢?
d
dalianyin
小厂来的一般快操猛
f
feicui
一个用户 发表于 2024-10-10 07:19
第一位大侠是天竺高手吗?
我猜他会说他上家公司有个QA team。

应该哪的人都有,中国人里也有做完了等别人告诉有问题再改的
L
Littlepony2020
没有专门测试的人?我们开发测试1:1
V
Vorlon
看来这儿程序员还是多。。。不管是楼主还是楼里的人,都觉得楼主说的这啥做unit test是常识,后面具体的push code啊啥的也是常识,所以根本不需要解释。

要是来个医生律师啥的,说自个儿招的人某个专业操作没做,估计很多人就囧逼了。。。这操作是啥?
c
coalpilerd
你们push code不需要reviewer approval?
l
liquidclem
Helen_Wood 发表于 2024-10-10 05:35
招了两个人,其中一个人,代码写完了,从来不做unit test, 文档中的5个需求,做完3个,就开始提交代码,然后开始刷手机,
我还得帮他测试,发现需求少了2点没做,发邮件,然后他做完一个需求,又开始push代码,
我把代码pull下来一跑,10个错误,然后找他,他立即改完,又push上去,我又得pull下来,一跑,还有5个错误没改。
最后发现还有一个需求没做,找到他,他就开始question 这个需求不合理,用户根本不会用到。
招了他简直,累死了, 但凡他写完了,自己花30秒测试一下,都会省去其他人半个小时的时间。
另外一个下属就非常好,写完代码,自己做好unit test,逐项检查开发文档,确认无误才push代码,从来没任何问题。

看不懂, 什么push pull的
z
zZZk
D
Deja_Wu
Littlepony2020 发表于 2024-10-10 09:44
没有专门测试的人?我们开发测试1:1


Unit test 是developer 的活,不是QA的活。但是没有reviewer approval是怎么merge into main branch 的?
y
yoyo2007
看来是用git review不就是防止出现这种情况? 也许需要建一个feature branch来跑一些jobs 来控制merge到main branch。
王力宏
review 不是防止这个的,各种 precheck 是防止这个的。
j
jeso1
楼主太菜。这种还留着过年
淘气小丸子
回复 20楼 jeso1 的帖子
赞同层主
j
juliadjh
楼上正解,ci/cd 会直接fail。或者你不approve PR呗,反正他在feature ,也不影响dev/main
m
mfc
这种有严重隐患的比摸鱼可怕多了。
y
yomayo131323
Helen_Wood 发表于 2024-10-10 05:35
招了两个人,其中一个人,代码写完了,从来不做unit test, 文档中的5个需求,做完3个,就开始提交代码,然后开始刷手机,
我还得帮他测试,发现需求少了2点没做,发邮件,然后他做完一个需求,又开始push代码,
我把代码pull下来一跑,10个错误,然后找他,他立即改完,又push上去,我又得pull下来,一跑,还有5个错误没改。
最后发现还有一个需求没做,找到他,他就开始question 这个需求不合理,用户根本不会用到。
招了他简直,累死了, 但凡他写完了,自己花30秒测试一下,都会省去其他人半个小时的时间。
另外一个下属就非常好,写完代码,自己做好unit test,逐项检查开发文档,确认无误才push代码,从来没任何问题。

不是有coverage测试的东西吗?没有一定的coverage就fail the build.
j
jiangyoudang17
document一切。这些人就是态度问题,总觉得就给我这点钱,还想让我好好干活的心态。能踢走就早点踢走