每次写完代码提交的时候都得经过pm的审核吗?

蓝纸汉 发布于 2016/01/22 10:50
阅读 3K+
收藏 0

我每次提交完代码后,pm只要一更新代码就会把我叫过去,问我为什么要这么改,你能确保没有问题吗等。。。。

每一次都问,有那么一俩次他忙的时候侥幸逃过了

我想问一下,你们也这样吗,我被这么的不行了

以下是话题补充:

@蓝纸汉:刚刚又叫我问我为什么要在model里面多加一个字段 (2016/01/22 11:07)
加载中
1
魔力猫
魔力猫

我感觉这不是代码审核,而是故意找茬,你没得罪他吧。

如果是正规代码审核,应该是定期对所有人的提交代码进行评审,而且评审者与被评审者是平等的,双方是在讨论代码如何能否更优秀,而是居高临下进行审判。

1
叫我刀刀
叫我刀刀
其实真相是   他喜欢你
0
bobdog1986
bobdog1986
赶紧局域网上个gogs,用go 写的git server。这样你从他的项目fork下来,每次提交完给他merge request,他同意就会合并了。可以比较代码以及看提交comments,大大减少沟通工作。
聽雨人
聽雨人
回复 @蓝纸汉 : 他不是pm,而是pm加项目经理吧。
bobdog1986
bobdog1986
回复 @滔哥 : 我搞c#也就1年,用git.osc,这玩意真的是太省心了。
滔哥
滔哥
可能他不懂,在向你们学习
bobdog1986
bobdog1986
回复 @蓝纸汉 : 搞个git吧,svn太反动了~
蓝纸汉
蓝纸汉
我们用的svn,我们公司有3个开发,一个pm,pm基本不敲代码
0
开源中国创始人
开源中国创始人
为什么加我们是问的,但是不问有没有问题
0
yongqqq
yongqqq
没有 你们代码严格啊·好事!
0
coin_
coin_
既然是PM,我觉得就应该做到这一点,要熟悉代码各个角落,对项目负责
0
senierr_zhou
senierr_zhou
正常的CodeReview,应该是:有不合理处,直接指出不合理处,若无则通过。每次都要解释为什么要这么做,没必要,也浪费了各自很多时间。如果一直这样,是很不可思议的一件事。。。(或者只是他太闲,想找个人墨迹墨迹?)
0
天使broken
天使broken
没有  随时都可以提交,但是前提是不要影响到他人的东西,不过建立了分支也不会存在这种情况吧
0
小黑001
小黑001
这样的pm才让他的上级放心,很难得
自由之信
自由之信
回复 @蓝纸汉 : 至于他问你是否确保没问题,不妨告诉他,这个修改不会带来新的或者是regression bug,如果出现bug,应该是前面的代码的问题就好了,因为确保没问题,是很难回答的,不可能没问题,只是问题不出在修改上就ok了
自由之信
自由之信
回复 @蓝纸汉 : 哈哈,如果你觉得pm会去测试,那就错了,pm怎么会像程序员一样去测试代码什么的,要不然要程序员干什么,pm就是负责给客户看app的时候,没问题就ok了。习惯一下吧,我们的心理作祟就是如果pm问了,就是对我们不信任,或者是觉得我们代码写的不好,其实pm只是为了保证代码别出什么问题,到了给客户交代的时候,结果出现bug什么的。
bobdog1986
bobdog1986
回复 @蓝纸汉 : 说实话,真的是100%有必要讲明白为什么这么改,而且最好还得留有记录,否则年深日久了,过上3个月,你可能都不知道为什么这么改了,要是再换个人,更没法接手了
蓝纸汉
蓝纸汉
我觉得pm不放心的话,可以自己多测两遍,没有必要让人讲一遍为什么要这样做,什么地方改过了,为什么要改, 至于上级放心,确实我们上级想当的放心,给予了pm足够的信任
返回顶部
顶部