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

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

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

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

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

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

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

© 2021 V2EX