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