看到同事的 PR 里搞手动对齐,还进了 master,比吃口 shi 还难受

2024-08-15 15:15:48 +08:00
 wanmyj
加一个枚举变量,结果名字太长,重新对齐改了几十行,加我 review 的时候我给提了 comment ,要她缩短变量名,不要搞手动对齐,然后就没管了,毕竟我不是 commiter ,只有 approve 权限,没 merge 权限。

今天我要把她和几个人之前提交的变量合并到一个结构体里,结果看记录,瞬间被喂了一口 shi ,日麻整个枚举类型全都是她提交的记录,还得回退一个 MR 才能看到哪些是真正她提交的。
4692 次点击
所在节点    程序员
40 条回复
Jinnrry
2024-08-15 17:25:27 +08:00
@wanmyj #12 这都习惯了,下次遇到问题的时候你也装傻,直接找到她,xxx ,你看下这里啥问题呢,我看记录是你改的。
GrayXu
2024-08-15 17:37:50 +08:00
@wanmyj #3 他说的没问题,就是你们 git 流程的问题,这种就给自动化做
crysislinux
2024-08-15 17:43:58 +08:00
我们项目五六年了,现在有人力主引入 prettier 来格式化,后果就像你说的,git blame 废了一大半。。
Zenon
2024-08-15 17:46:27 +08:00
@wanmyj #12 卧槽,看你描述我都开始生气了
hxndg
2024-08-15 17:47:41 +08:00
@wanmyj 还是没懂,如果调用某个 bot 或者统一 format 不是一样得会退版本吗?这个不能直接 cherrypick 是吗?
AirCrusher
2024-08-15 18:03:17 +08:00
andforce
2024-08-15 19:27:26 +08:00
fmt 得在工程一开始就得有
1xiaozi
2024-08-15 19:46:49 +08:00
@wanmyj 遇到过一样的同事。想请问下是怎么提交会导致这种情况。
jeesk
2024-08-15 20:11:50 +08:00
@AirCrusher 改动过的才允许格式化会减少很多坑
cheetah
2024-08-15 20:21:16 +08:00
@crysislinux 还是好处大于坏处的。而且还有 #26 说的 .git-blame-ignore-revs
crysislinux
2024-08-15 20:28:03 +08:00
@cheetah 没早发现这个,不然让他们一次性格式化了。现在是改到哪个文件格式化哪个。
cheetah
2024-08-15 20:33:51 +08:00
@crysislinux 这太难受了,赶紧格式化一次吧
yb2313
2024-08-15 20:35:29 +08:00
所有地方他都动了, 那问题都找他解决
wanmyj
2024-08-15 21:10:57 +08:00
@MrSheng yes ,引入的 git 记录非常不方便。
@hxndg "如果调用某个 bot 或者统一 format 不是一样得会退版本吗?" --- 其实 bot 的记录也是蛮讨厌,但最起码能看到修改人是 bot ,不至于分不清。和 cherrypick 关系不太大。
@1xiaozi “怎么提交会导致这种情况” --- 没太明白你想问什么
wanmyj
2024-08-15 21:15:00 +08:00
@AirCrusher 确实是个神器,但我看了下,还不太能用,她是 formating 和 editing 放在一个 commit 提交上去了,这个 commit 如果 ignore 了又会导致正式修改的 blame 受影响。So~ fine...
1xiaozi
2024-08-15 21:16:19 +08:00
@wanmyj 就是 [她的行为导致了 git blame 的返回结果是每行代码的修改人都是她。] 这个情况
1xiaozi
2024-08-15 21:17:36 +08:00
@1xiaozi 补充一下、正常使用 add/commit ,merge ,rebase 都没出现过这个现象。但是第一次看到都震惊了、不知道怎么做到的
wanmyj
2024-08-15 21:55:15 +08:00
@1xiaozi 这个情况和 git 命令没关系,单纯就是因为她修改了那些本不应该碰的代码,并提交了这些修改。
ffw5b7
2024-08-16 10:31:27 +08:00
是习惯吧,我每次改完代码都会 ctrl s;ctrl l.保存加格式化。
话说代码不格式化不难受吗?
不是没有手动对齐吧。
jwj
2024-08-16 10:52:25 +08:00
这都小儿科

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/1065240

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX