这是一个创建于 421 天前的主题，其中的信息可能已经有所发展或是发生改变。
加一个枚举变量，结果名字太长，重新对齐改了几十行，加我 review 的时候我给提了 comment ，要她缩短变量名，不要搞手动对齐，然后就没管了，毕竟我不是 commiter ，只有 approve 权限，没 merge 权限。
今天我要把她和几个人之前提交的变量合并到一个结构体里，结果看记录，瞬间被喂了一口 shi ，日麻整个枚举类型全都是她提交的记录，还得回退一个 MR 才能看到哪些是真正她提交的。
今天我要把她和几个人之前提交的变量合并到一个结构体里，结果看记录，瞬间被喂了一口 shi ，日麻整个枚举类型全都是她提交的记录，还得回退一个 MR 才能看到哪些是真正她提交的。
 | 1 luckyc 2024-08-15 15:23:25 +08:00 走一个就行了 |
 | 2 MoYi123 2024-08-15 15:28:39 +08:00 有 review 机制, CI 里没有 fmt 吗? 如果符合 fmt, 手动对齐又有什么影响? |
 | 4 maxwellz 2024-08-15 15:37:51 +08:00 逆天... |
 | 5 cxsz 2024-08-15 15:39:30 +08:00 可以统一配置下 git hooks 在 commit 前格式化代码，或者加到 ci 里面 |
 | 6 bugmaker233 2024-08-15 15:42:17 +08:00 没看懂你在说什么.....又不是不能用.jpg |
 | 7 kokutou 2024-08-15 16:04:35 +08:00  1 还是工作量不够大 居然还有时间强迫症对齐 |
 | 8 davin 2024-08-15 16:20:18 +08:00 push 之前 husky + 各种 auto fix 走一遍先，书同文，车同轨。避免占用额外的开发时间 |
 | 9 C4D4zRNpq9vFSlJW 2024-08-15 16:21:25 +08:00 1 关键词：《她》 |
 | 10 mcfog 2024-08-15 16:27:21 +08:00 不是 ignore whitespace 吗 |
 | 11 ih8es9OIzne0959p 2024-08-15 16:27:34 +08:00 没看懂，能举个例子吗 op ，属实纳闷 |
 | 12 wanmyj OP @ajaxgoldfish 没看懂说明这个做法没太影响到你。 简单说，git 有个 blame 功能，能直观看到每行代码的上次修改人。她的行为导致了 git blame 的返回结果是每行代码的修改人都是她。 |
| 13 ih8es9OIzne0959p 2024-08-15 17:11:26 +08:00 via Android @wanmyj 明白了。多谢。 |
 | 14 liwenka1 2024-08-15 17:12:55 +08:00 @ajaxgoldfish 前期无所谓，你想想后期你先看看这个功能是谁写的，然后发现全是一个人写的，这不是玩呢吗？问谁都找不到人 |
 | 15 zhtyytg 2024-08-15 17:14:58 +08:00 @2020583117 倒也不是找不到，而是每次都要翻文件修改记录，很烦 |
 | 16 sks4728 2024-08-15 17:16:09 +08:00 不是不能用， 用不了 就走一个 |
 | 17 mogita 2024-08-15 17:20:06 +08:00 她喂你 shi ，你赏她饭（ |
 | 18 nyxsonsleep 2024-08-15 17:21:49 +08:00 idea 可以直接按行查看多次提交的历史 |
 | 20 LieNoWell 2024-08-15 17:23:25 +08:00 A 改动了代码 3-5 行，B 现在需要修改第 2 行，但是因为额外的对齐操做而动了 3-5 行 。这导致 blame 的时候看到的 2-5 行都是 B 进行操作的，是这个意思吗？ |
 | 21 Jinnrry 2024-08-15 17:25:27 +08:00 3 @wanmyj #12 这都习惯了，下次遇到问题的时候你也装傻，直接找到她，xxx ，你看下这里啥问题呢，我看记录是你改的。 |
 | 23 crysislinux 2024-08-15 17:43:58 +08:00 via Android 我们项目五六年了，现在有人力主引入 prettier 来格式化，后果就像你说的，git blame 废了一大半。。 |
 | 25 hxndg 2024-08-15 17:47:41 +08:00 @wanmyj 还是没懂，如果调用某个 bot 或者统一 format 不是一样得会退版本吗？这个不能直接 cherrypick 是吗？ |
 | 26 AirCrusher 2024-08-15 18:03:17 +08:00 5 你需要.git-blame-ignore-revs： https://gist.github.com/kateinoigakukun/b0bc920e587851bfffa98b9e279175f2 |
 | 27 andforce 2024-08-15 19:27:26 +08:00 via iPhone fmt 得在工程一开始就得有 |
 | 28 comeAcross 2024-08-15 19:46:49 +08:00 via Android @wanmyj 遇到过一样的同事。想请问下是怎么提交会导致这种情况。 |
 | 29 jeesk 2024-08-15 20:11:50 +08:00 via Android @AirCrusher 改动过的才允许格式化会减少很多坑 |
 | 30 cheetah 2024-08-15 20:21:16 +08:00 @crysislinux 还是好处大于坏处的。而且还有 #26 说的 .git-blame-ignore-revs |
| 31 crysislinux 2024-08-15 20:28:03 +08:00 @cheetah 没早发现这个，不然让他们一次性格式化了。现在是改到哪个文件格式化哪个。 |
| 32 cheetah 2024-08-15 20:33:51 +08:00 1 @crysislinux 这太难受了，赶紧格式化一次吧 |
 | 33 yb2313 2024-08-15 20:35:29 +08:00 所有地方他都动了, 那问题都找他解决 |
| 34 wanmyj OP |
| 35 wanmyj OP @AirCrusher 确实是个神器，但我看了下，还不太能用，她是 formating 和 editing 放在一个 commit 提交上去了，这个 commit 如果 ignore 了又会导致正式修改的 blame 受影响。So~ fine... |
| 36 comeAcross 2024-08-15 21:16:19 +08:00 @wanmyj 就是 [她的行为导致了 git blame 的返回结果是每行代码的修改人都是她。] 这个情况 |
| 37 comeAcross 2024-08-15 21:17:36 +08:00 @1xiaozi 补充一下、正常使用 add/commit ，merge ，rebase 都没出现过这个现象。但是第一次看到都震惊了、不知道怎么做到的 |
 | 39 ffw5b7 2024-08-16 10:31:27 +08:00 via Android 是习惯吧，我每次改完代码都会 ctrl s;ctrl l.保存加格式化。 话说代码不格式化不难受吗？ 不是没有手动对齐吧。 |
 | 40 jwj 2024-08-16 10:52:25 +08:00 这都小儿科 |
Select Language