组里 40 老大哥有错不改还注释他人代码怎么解

[bingyiyu]

spring cloud 项目，用的 feign，框架是老大哥搭的。前阵子测试给我提 bug 说异常了代码还继续执行。 我就查了一下加了个全局的 decode，但是要求返回值都是统一的自定义 Respose 对象，拦截器里拿这个对象去反序列化。我们的代码绝大部分是标准返回自定义 Respose 对象的，但是老大哥偷懒直接返回了 VO 未拿 Respose 对象包装导致反序列化失败。 我给提了我去全局扫一遍帮他改，支支吾吾说要报告领导（产品还没上线而且又不牵扯业务逻辑），然后加了个 ifelse 把我的逻辑绕过去了(还没绕对，我 debug 一看还是有问题)。 然后就进入了扯皮阶段。好说歹说领导给同意我的做法了。我就把她的 ifelse 去掉了。提交至 master 。 然后第二天骚气的来了，前端给我提 bug 。 我一看好家伙他直接把我的拦截器给注释掉了，导致 feign 异常未捕获继续执行了

[dayeye2006199]

建议先对代码 review，分支合并先有一套规范再说。直接往 master 上面推代码怎么看都不是个好事儿

[auh]

@no1xsyzy 都行

[crclz]

@bingyiyu 那你就让他在 master 写好了，估计他也不会使用除了 commit 和 push 以外的功能。你把发布分支换为另一个分支，例如 release 。

[crclz]

@360511404 其实小团队可以采用 trunk-based，即大家都在 master 上面写代码，优势是保证持续集成（集成=代码合并+单元测试+xx 测试），利于防止那种几周都不合并的代码。

[vone]

1 、你是刚 node 转 java
2 、基础框架搭建是这位老员工的负责，非你的职责范围
3 、你在没有和上级沟通的前提下修改了基础框架，在全局范围内产生了破坏性升级
4 、修改后可能没有进行汇报和宣导（老员工不知情）
5 、过于看重技术，对其他同事缺少尊重

[IamUNICODE]

我一个人也不在 master 上改代码，分出一个 dev，分出一个自己，会污染自己代码的特殊功能再分一个

[jones2000]

沟通下, 不能协商的话,就根据的原来的坑继续写. 前人挖的坑, 你不用去填, 继续往下挖, 挖到不能挖的时候自然有人给你填坑你的, 有大佬在前面挡子弹怕什么, 继续挖.

[bingyiyu]

今天老哥又往 master 上提未编译通过的代码了。忍不了，部门群里 @他和领导了。
我都给在项目里配了 githook push 自动编译都不用，这事情也不是第一次了
master 上不给直接提交的事情也提过好几次了
老顽固无疑了

[stillyu]

@X0ray 不是我哦，我今年 30 岁，怕是完成不了原始积累了。上家公司一个同事，就是 40 岁，上海两套房，杭州一套房。来上班就是混的