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

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

今天我要把她和几个人之前提交的变量合并到一个结构体里,结果看记录,瞬间被喂了一口 shi ,日麻整个枚举类型全都是她提交的记录,还得回退一个 MR 才能看到哪些是真正她提交的。
4686 次点击
所在节点    程序员
40 条回复
l4ever
2024-08-15 15:23:25 +08:00
走一个就行了
MoYi123
2024-08-15 15:28:39 +08:00
有 review 机制, CI 里没有 fmt 吗? 如果符合 fmt, 手动对齐又有什么影响?
wanmyj
2024-08-15 15:36:10 +08:00
@MoYi123 我在说 git 提交记录被无意义的多刷一遍,跟 fmt 有什么关系?
maxwellz
2024-08-15 15:37:51 +08:00
逆天...
cxsz
2024-08-15 15:39:30 +08:00
可以统一配置下 git hooks 在 commit 前格式化代码,或者加到 ci 里面
bugmaker233
2024-08-15 15:42:17 +08:00
没看懂你在说什么.....又不是不能用.jpg
kokutou
2024-08-15 16:04:35 +08:00
还是工作量不够大
居然还有时间强迫症对齐
davin
2024-08-15 16:20:18 +08:00
push 之前 husky + 各种 auto fix 走一遍先,书同文,车同轨。避免占用额外的开发时间
C4D4zRNpq9vFSlJW
2024-08-15 16:21:25 +08:00
关键词:《她》
mcfog
2024-08-15 16:27:21 +08:00
不是 ignore whitespace 吗
ih8es9OIzne0959p
2024-08-15 16:27:34 +08:00
没看懂,能举个例子吗 op ,属实纳闷
wanmyj
2024-08-15 17:04:43 +08:00
@ajaxgoldfish 没看懂说明这个做法没太影响到你。

简单说,git 有个 blame 功能,能直观看到每行代码的上次修改人。她的行为导致了 git blame 的返回结果是每行代码的修改人都是她。
ih8es9OIzne0959p
2024-08-15 17:11:26 +08:00
@wanmyj 明白了。多谢。
liwenka1
2024-08-15 17:12:55 +08:00
@ajaxgoldfish 前期无所谓,你想想后期你先看看这个功能是谁写的,然后发现全是一个人写的,这不是玩呢吗?问谁都找不到人😁
zhtyytg
2024-08-15 17:14:58 +08:00
@2020583117 倒也不是找不到,而是每次都要翻文件修改记录,很烦
sks4728
2024-08-15 17:16:09 +08:00
不是不能用, 用不了 就走一个
ck65
2024-08-15 17:20:06 +08:00
她喂你 shi ,你赏她饭(
nyxsonsleep
2024-08-15 17:21:49 +08:00
idea 可以直接按行查看多次提交的历史
Donaldo
2024-08-15 17:22:17 +08:00
@wanmyj #3 他意思应该是如果有 fmt 可能就不会出现需要手动对齐的问题了
MrSheng
2024-08-15 17:23:25 +08:00
A 改动了代码 3-5 行,B 现在需要修改第 2 行,但是因为额外的对齐操做而动了 3-5 行 。这导致 blame 的时候看到的 2-5 行都是 B 进行操作的,是这个意思吗?

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

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

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

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

© 2021 V2EX