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

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

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

简单说,git 有个 blame 功能,能直观看到每行代码的上次修改人。她的行为导致了 git blame 的返回结果是每行代码的修改人都是她。
ajaxgoldfish
34 天前
@wanmyj 明白了。多谢。
2020583117
34 天前
@ajaxgoldfish 前期无所谓,你想想后期你先看看这个功能是谁写的,然后发现全是一个人写的,这不是玩呢吗?问谁都找不到人😁
zhtyytg
34 天前
@2020583117 倒也不是找不到,而是每次都要翻文件修改记录,很烦
sks4728
34 天前
不是不能用, 用不了 就走一个
ck65
34 天前
她喂你 shi ,你赏她饭(
nyxsonsleep
34 天前
idea 可以直接按行查看多次提交的历史
Donaldo
34 天前
@wanmyj #3 他意思应该是如果有 fmt 可能就不会出现需要手动对齐的问题了
MrSheng
34 天前
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