1
roundRobin 159 天前 2
这些应该写在 merge request 的 description 里面,而不是 commit message
|
2
netabare OP @roundRobin 对一个 PR 来说,这个 algorithm 可能只是其中一小部分,issue 里面提到的东西经常需要改蛮多东西的,而且产品那边也可能也会反复需求。
|
3
roundRobin 159 天前
@netabare 那这种情况一个 issue 应该分成多个 PR 来提交,每个做其中一部分功能,PR 的可读性,单元测试的可靠性都能增加,然后一起做个 e2e testing 。 我 review 的时候超过一页的 pr 都是直接 decline 让他 split 之后再提交
|
4
arischow 159 天前
看组织/团队或者项目要求,各有优缺
|
5
wu67 159 天前 2
我直接把 commit message 当月报来写. 到交月报的时候, 直接 git log 格式化一下就完事
|
6
Contextualist 159 天前 1
还是得拆分
commit message 长是表象,commit 过大或许才是根本原因。 > 总是担心意图没解释清楚别人难以理解自己的代码。 类似的,如果你也倾向于写大段的注释,不妨停下来想想看实现是不是可以被拆成多个函数/类 不用总想着一次写好一个完整的 commit 写一个片段就可以做一个临时的 commit ,然后在 push 前把这些临时的 commits 重新排列与合并成最终的 commits 善用 git rebase -i |
7
ytmsdy 159 天前
不是挺好的么,总比 fix bug 强!
|
8
yKXSkKoR8I1RcxaS 159 天前 2
@ytmsdy 1111111111 ,Update ,修改,回家再搞
|
9
tolbkni 159 天前
你说的这些内容以前都是放在代码里作为注释块的
|
10
GeruzoniAnsasu 159 天前
> 发 PR 的时候好像我总喜欢写得像 GitHub 的 readme 一样
是对的 > 长 commit 是新手的普遍操作,senior 的 commit 普遍一句话带过 可能是因为你没能很好地贯彻 1 commit 只做一件事的原则 |
11
xubingok 159 天前
1.只写你做了什么即可,不用写做这个的原因和具体方式.
2.示例 commit 明显可以拆成几个,典型的"删除过期代码". |
12
Scarb 159 天前 1
之前看到 netty 的三段式 commit message ,觉得挺好,可以参考一下。举个例子
``` Aggressively remove PoolThreadCache references from its finalizer object Motivation: If a cache's FastThreadLocalThread owner win the race to remove the cache, due to debugging capabilities, it's finalizer will still retain a strong reference to it, causing few classes to leak (and eventually, their ClassLoader). Despite we cannot avoid finalizers to wait the finalization pass, we can reduce the memory footprint of "leaked" instances before the finalization happen. Modification: non-debug early cache removal can remove the cache strong reference within FreeOnFinalize, making it an emtpy shell, eligible for GC. Result: Smaller memory footprint while waiting finalization to happen ``` |
13
guanzhangzhang 159 天前
commit 写表象,注释写细致
|
14
tyrantZhao 159 天前
这个多半是在一个 commit 里做了太多事了,不太合适。
|
15
janus77 159 天前
问一个问题,你大段覆盖别人的代码的时候,会看别人之前的 commit msg 吗?会担心看不懂而影响你的覆盖工作吗?
|
16
linzhe141 159 天前
@tyrantZhao 不是自己项目,真不想弄那么细
|
18
intouchables 159 天前
|
19
bigfei 159 天前
直接 copilot 生成 commit msg 就可以了
|
20
Daniel17 159 天前
挺好的,我同事的 commit message 都是 update ,update
|
21
echoless 159 天前
|
22
shaozelin030405 159 天前
update ,fix ,fix: 调整
|
23
TeslaM3 159 天前
挺奇葩的,如今 IDEA 这么丰富的现状还使用命令。
|
24
feiyan35488 159 天前
业务需求和设计思路可以写到 mr 的 message 里,
算法的文档可以写到代码注释里 |
25
feiyan35488 159 天前
@TeslaM3 命令行 效率高,与 idea 无关,可以避免脱离 idea 不会使用 git 的情况,比如 linux 上,或换 idea 的情况
|
26
woshihgs 159 天前
牛 四年级就开始做项目了
|
27
1rv013c6aiWPGt24 159 天前 via Android
四五年级…牛逼啊
|
28
RockShake 159 天前
你的内容适合放在 Pull Request 里面,Commit 的规范可以参考 Conversional Commit ,那个说的挺清楚的
|
29
artiga033 159 天前 via Android
commit 我一般是保证简洁易读,甚至有时为了压缩长度在保证可读性的前提下破坏自然语言语法(比如省略一些虚词介词量词)
|
31
netabare OP @UncleCAT4 是指大四大五啦,法国的大学学制是五年的。
@roundRobin 主要也是看团队里面并没有把 PR 细化到这个地步,一般一个 issue 就是一个 PR ,如果大家的 commit 和 PR 都很细的话我肯定是跟着他们做的。GitHub 上的个人项目倒是不会做那么细就是…… @Contextualist 好建议,我倒是还没太熟悉 rebase 的用法,感谢提醒。 @tolbkni 注释我感觉问题在于后面改来改去的很容易就会丢掉上下文…… @GeruzoniAnsasu 示例这个确实不太好,不过一方面也有看团队里一个 PR 也就几个 commit ,我自然会倾向于避免上来就推几十个 commit 的巨型 PR ,所以也会有想要把 B 「虽然是不同的事情但在功能上相互联系的几件事组合在一起」的倾向,尤其是发了两次包含连续四五个 Refactor 的 PR 之后。 @Scarb 这个模版倒是不错,有时候我感觉我也有点像在往这方面靠。 @janus77 会的,会先看 git blame 找出 commit 的历史记录,问题是一般来说别人都是简单的一句话甚至没有提到他的代码意图,所以我才感觉如果有个 commit message 会清晰很多。 |