这是一个创建于 1774 天前的主题，其中的信息可能已经有所发展或是发生改变。
最近入职一家新公司，然后这边提交代码每次提交都有人审核。比如我修改了某个位置，优化了执行的逻辑，或者注释掉了某段不需要的代码，审核人告诉我这样不行，需要还原，我当时很气，感觉这公司太僵化，甚至想马上离职。
我这种想法对吗？还是很多公司都是这种流程
 |
1
qq8758173803 2020-01-18 08:43:51 +08:00
如果薪水合适，这种流程便不算啥。我们小公司，开发就没那么多麻烦事了。
|
 |
2
guyeu 2020-01-18 10:39:12 +08:00
reviewer 告诉你不行肯定会告诉原因的，如果你觉得不合理可以刚啊。。
|
 |
3
lcsoft 2020-01-18 11:14:48 +08:00
这流程看着没什么问题。
|
 |
4
orzorzorzorz 2020-01-18 11:49:29 +08:00
从字面看楼主表达的是，“我优化了代码，但老大不同意，很气，要离职”。
从实际想，楼主没给出优化前后的 benchmark，看上去也没深入研究原先代码具体在其他什么地方、多少地方用到了，也没给老大说这么改不行的原因。直接一句“好气啊”，就开始顺理成章地想着，“啊，这公司不值得我努力，骑驴找马骑驴找马”，然后开始了摸鱼生活。 我懂，我懂 :doge |
 |
5
waterjiao 2020-01-18 12:39:16 +08:00
code review 难道不会提出来吗？
|
 |
6
xxxy 2020-01-18 14:21:02 +08:00
这是正常流程，反而是让你随意改代码的公司不靠谱
|
 |
7
flyico 2020-01-18 16:48:38 +08:00
一些大公司 PR 提出后修修改改，一周后才 merge，这都是常事
|
 |
9
wangyzj 2020-01-18 21:30:24 +08:00
这个不是 git 流程的问题
这个是管理流程的问题 |
 |
10
sungykk 2020-01-19 00:31:47 +08:00
这不很正常吗？ 我们是要修改任何代码都需要 jira 上提 bug/feature id ---> git push 到 gerrit，组内至少一个人 review +1 , QA 测试 之后 verify , 总的代码管理人 review +2 之后才能 merge .
|
 |
11
liaojl 2020-01-19 09:02:35 +08:00 via Android
这才是正常流程吧
|
 |
12
cco 2020-01-19 09:10:02 +08:00
用优化过后的效果说服他不就行了：
性能更好了？解耦了？可复用性强了？代码更短了？更健壮了？ 如果有，他不通过那奏是他的问题，否则。。 |
 |
13
Greendays 2020-01-19 09:30:48 +08:00
就算你的优化确实提高了性能，但是如果没有这个需求的话，我也不会不会让你通过的。这么努力干什么，下次有需求了再提交不好吗（
|
 |
14
wwwwzf 2020-01-19 09:40:05 +08:00
适应它。
你就是英雄。 |
 |
15
xhf1024 2020-01-19 09:47:25 +08:00
@orzorzorzorz 哈哈哈现在推理哈哈哈
|
 |
16
chitanda 2020-01-19 10:36:18 +08:00
我很羡慕这样的流程
|
 |
17
msg7086 2020-01-19 10:37:39 +08:00
你还是离职吧，找一家没有代码审核的公司，最好之前的程序员什么辣鸡代码都往库里灌，灌完就走人，让你接手来维护。
|
 |
18
Uyuhz 2020-01-19 10:53:47 +08:00
我很羡慕这样的流程+1。基本上一直单干，想找个人给我 code review 都没有
|
 |
19
atonku 2020-01-19 10:55:39 +08:00
add->commit->push
|
 |
20
971586331 2020-01-19 15:22:54 +08:00
有人看你的代码，这样编程水平才能提高啊，我们公司的代码都没人看，写了几年都是这个水平
|
 |
21
Tn5ohB1Yecdk3qCK 2020-01-19 15:27:00 +08:00
这种流程很赞
|
 |
22
linZ 2020-01-19 16:09:42 +08:00
这是在保护你， 保证你少返工，不知足么
|
Select Language