b**********h
发帖数: 419 | 1 印度老板印度组
老板给我一个小feature 的任务，我check in后，老板指派另一个烙印code review。
挑出了一堆麻烦的问题，但这些都不是我添加的代码，而是之前印度老板学写java时的
新手错误
这种情况如何处理好 |
w***t
发帖数: 1474 | 2 在烙印组你还想怎样。老板拉的这坨屎谁都不愿擦，最后肯定是落到老中头上。要么接
受现实把屎擦干净，做个听话的小弟，说不定还能融入烙印圈子。要么换个非烙印组。
【在 b**********h 的大作中提到】
: 印度老板印度组
: 老板给我一个小feature 的任务，我check in后，老板指派另一个烙印code review。
: 挑出了一堆麻烦的问题，但这些都不是我添加的代码，而是之前印度老板学写java时的
: 新手错误
: 这种情况如何处理好
|
d******b
发帖数: 73 | 3 首先 要想到，把工作做好。
然后 git blame 回信 抄送 所有人 |
u********s
发帖数: 1047 | 4 都chekc in了还review个啥。不都是先review才check in |
h**c
发帖数: 1979 | |
h**c
发帖数: 1979 | 6 会不会说你见到错误代码没有改正，所以还是你的问题？
【在 h**c 的大作中提到】
: 这么搞笑，代码谁写的还不知道吗
|
b**********h
发帖数: 419 | 7 烙印满嘴都是伟光正，一定会这样说
: 会不会说你见到错误代码没有改正，所以还是你的问题？
【在 h**c 的大作中提到】
: 会不会说你见到错误代码没有改正，所以还是你的问题？
|
b**********h
发帖数: 419 | 8 烙印满嘴都是伟光正，一定会这样说
: 会不会说你见到错误代码没有改正，所以还是你的问题？
【在 h**c 的大作中提到】
: 会不会说你见到错误代码没有改正，所以还是你的问题？
|
z*********n
发帖数: 1451 | 9 这都什么乱七八糟的，先checkin再review？review时不会diff下哪些是你的代码？
让你改不是你写的代码这种，标准流程不是，
1,先建立bug做记录。否则你有什么理由碰这些运转了多年的代码？你认为它有问题，
那建立bug。
2.Investigate bug，证明这些代码确实有问题。多少年没碰也没出问题，那没有必要
修改，万一改出bug呢？
3.Blame看是谁的代码，决定把bug assign给谁。
让你修bug，你把标准流程走完先，看老印啥反映。 |
w*****g
发帖数: 1415 | 10 Tell your boss that it may not worth the risk since you are not sure what
exactly it does and there is no bug with it so, |
U********I
发帖数: 20 | 11 Tell your boss that it may not worth the risk since you are not sure what
exactly it does and there is no bug with it so
-------
这么说的话，老板会不会觉着LZ能力不行，然后排挤LZ。
不过讲真，这种组和老板的话，早晚屎盆子要扣下来。
支持 git blame 回信 抄送 所有人。 |
k*********s
发帖数: 4474 | 12 肯定是你来改，没跑
查history找到谁commit
然后说你需要3天改代码2天测试，这礼拜做不了别的活儿了
【在 b**********h 的大作中提到】
: 印度老板印度组
: 老板给我一个小feature 的任务，我check in后，老板指派另一个烙印code review。
: 挑出了一堆麻烦的问题，但这些都不是我添加的代码，而是之前印度老板学写java时的
: 新手错误
: 这种情况如何处理好
|
P***t
发帖数: 165 | 13 如果和你的feature没有直接的关系，明确的回应都是很老的code，不是这个feature的
一部分，让他新建一个bug来cleanup这个烂东西，assign给你的老板，让他再去分配和
要求QA的资源。
不要让他把两个东西搅和在一起，造成问题是你引起的印象。 |
