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