代码提交指导

开发人员通过git commit和gitee pr的形式向openEuler Embedded进行贡献,其中git commit msg需遵循相应的规范。

openEuler Embedded部署了基于 gitlint 的机制来检查git commit是否符合规范, 具体的规范实现位于 代码仓 下的 .gitlintscripts/gitlint/openeuler_embedded_commit_rules.py 中。

commit msg样式与示例

commit msg提交规范样式如下(具体规则请看 详细说明 ):

header
<blank line>
body
<blank line>
footer
(cherry-pick info) #可选,若此commit通过cherry-pick获得,应注明来源commit id

完整示例1:

#此commit仅做示例,非真实commit
yocto: Update poky-4.0 grammer details

update yocto grammer: replace '_' with ':' for yocto version stability.

Signed-off-by: zhangsan <zhangsan@163.com>

完整示例2:

#此commit仅做示例,非真实commit
ros2: Fix ros2 sdk bugs for humble

* fix ament depend
* avoid pythonpath err in colcon
* fix find_path err in numpy for cross-compile

Fixes: adii232naidf(ros2: Upgrade ros2 sdk for humble)  #假定这是对应产生问题的commit,非真实commit
Closes: https://gitee.com/openeuler/yocto-meta-openeuler/issues/T7SQ40?from=project-issue   #假设这是要关闭的issue,非真实链接
Co-developed-by: zhangsan <zhangsan@163.com>    #共同贡献者
Signed-off-by: zhangsan <zhangsan@163.com>
Co-developed-by: lisi <lisi@163.com>            #共同贡献者
Signed-off-by: lisi <lisi@163.com>
Signed-off-by: lihua <lisi@163.com>             #提交人,共同贡献者之一

完整示例3:

#此commit仅做示例,非真实commit,实际版本更新内容未完全罗列
yocto!: Update poky version to 4.0(kirkstone)

* upgrade yocto version, because it can be maintained for longer.

* Here are the changes made during this upgrade:
  - some variables have changed their names.
  - because of the uncertainty in future default branch names in git repositories,
    it is now required to add a branch name to all URLs described by
    git:// and gitsm:// SRC_URI entries.
  - distutils has been deprecated upstream in Python 3.10 and thus the distutils* classes have
    been moved to meta-python. Recipes that inherit the distutils* classes should be updated to
    inherit setuptools* equivalents instead.
  - some recipes have been removed.
  - :append/:prepend in combination with other operators.

Link: https://docs.yoctoproject.org/migration-guides/migration-4.0.html
Signed-off-by: wangermazi <wangermazi@163.com>
(cherry picked from commit h84b88e9e49483aeec884am02ef98056622f6eb8)    #从其他分支cherry-pick过来的commit id信息

commit msg详细说明

  1. commit msg由三部分组成,分别为 headerbodyfooter ,每部分都不可少,且 headerbodybodyfooter 之间都必须有空行,整体commit msg使用英文描述。

  2. header 行必须有且仅有一行,形式为 <area>: <subject> , 在冒号后有一个空格。当提交为非revert类型时,行长度不超过80字符;为revert类型时,行长度不超过102字符,其中:

  • area 非空,一般为配方名称,或要更改的文件的简短路径,若同时修改多个配方,也可使用更抽象的属类,比如config, 层的名字,docs等。

  • subject 非空,应包含对更改的简介描述。必须使用祈使句和现在时态,即不可使用’changed’或’changes’,而是使用’change’(具体解释可见 )。不能在末尾加标点符号,不能包含表情符号,至少包含3个单词。

  • revert类型提交请看 第6点

  1. body 为提交的详细描述,非空。不能包含表情符号,并使用祈使句和现在时态。对于不包含URL链接的行,要求每行不超过100个字符,若包含URL链接,字符没有限制,不过建议将URL链接单独成一行,提高可读性。

  • 应包含改变的动机(why),改变的内容(what),怎样改变的(how),并将其与以前的行为进行对比。

  • 如果提交跨5个文件或者50行代码,应在 body 中描述出来,文档相关的提交除外。

  • 内容可引用之前commit信息, 记得同时带上commit id(SHA-1的前12个字符)和对应的 header 信息。

  • 可以使用连字符或星号表示更多段落,同时可以配合使用悬挂缩进,所有段落可适当用空行分开增强可读性。示例见 完整示例3

  1. footer 包含一个或多个标签,每个标签为一行,且标签之间不空行。 每行标签的形式为 <tag-name>: <tag-context> ,注意冒号后面有一个空格。 原则上每行不超过100个字符( Fixes 标签为不超过101字符),除非标签包含完整URL链接。另外 footer 必须包含 Signed-off-by 标签,且 footer 的末尾标签(最后一行)必须为 Signed-off-by 标签。以下为可使用标签的介绍:

Signed-off-by :社区不允许匿名贡献,每个commiter都必须进行身份确认(需要 签署CLA )。格式为:

#commiter-name为提交者的真实姓名拼音,如张三,李四 应分别为 zhangsan,lisi
#random@developer.example.org 必须为签署协议时账号绑定的邮箱
#注意commiter-name左右两边有空格
Signed-off-by: commiter-name <random@developer.example.org>

Closes :用于说明该次提交修复了 issue问题 ,标签后带上被修复的issue链接,由于issue位于openEuler社区,因此链接的前缀必须为:https://gitee.com/openeuler/ 。如果同时修复了多个issue,则可写多行 Closes 标签,每行带一个issue链接,如下所示:

Closes: https://gitee.com/openeuler/yocto-meta-openeuler/issues/I7SQ40?from=project-issue
Closes: https://gitee.com/openeuler/yocto-meta-openeuler/issues/I7071W?from=project-issue

Fixes :用于说明该次提交是为了解决之前 commit id 带来的问题,标签后附加之前产生问题的提交,包括其commit id(取SHA-1的前12个字符)和对应的header信息,如下所示(假设解决 完整示例2 带来的问题):

Fixes: 54a4f0239f2e(ros2: Fix ros2 sdk bugs for humble)

Co-developed-by :用于说明代码是由多个开发人员共同贡献,向共同作者提供归属(除了提交者本人)。每个 Co-developed-by 标签下都必须紧跟 Signed-off-by 标签,形式为(也可见于 完整示例2 ):

Co-developed-by: First Co-Author <first@coauthor.example.org>
Signed-off-by: First Co-Author <first@coauthor.example.org>
Co-developed-by: Second Co-Author <second@coauthor.example.org>
Signed-off-by: Second Co-Author <second@coauthor.example.org>

Link :用于使用附加背景和详细信息的网页对该次提交予以说明,此时,标签后接完整URL,示例如 完整示例3

  1. 若提交的代码包含了重大变更(如poky版本升级、重要接口参数减少、接口删除、迁移等),需在 header 行中的冒号前加一个 ,并且在 body 中完整描述重大变更信息(包括更改内容、原因及迁移位置等信息)。在 完整示例3 就使用了此方法。

  1. 如果是进行以往提交的回退,则 area 填写为revert, subject 为被回退提交的 commit id 和header信息,在 body 中应写回退的原因。比如要回退以上 完整示例1 (假设其commit id前12位为:54a4f0239f2e),则commit msg为:

revert: 54a4f0239f2e(yocto: Update poky-4.0 grammer details)

some reasons for revert.

Signed-off-by: developer <random@developer.example.org>
  1. 如果commit是通过 git cherry-pick 获得,建议使用 -x 选项(这样在cherry-pick时,git会自动生成来源说明),备注好源分支的commit id信息,便于后续追溯查看使用。正确格式为(也可见于 完整示例3 ):

#注意,该行必须位于Signed-off-by标签后,两者之间不留空行
#其中 6024a1fca1455a6e3dc84082f60bh2b481dc5f9e 指来源分支上的commit id
#使用”git cherry-pick -x“可以自动生成
(cherry picked from commit 6024a1fca1455a6e3dc84082f60bh2b481dc5f9e)
  1. 建议每次合并请求(Pull Request)不要超过10次提交。

本地检查commit msg

  1. 安装gitlint工具:

$ pip install gitlint
  1. 设置Commit-msg hook ,使得在每次提交时可按照代码仓根目录下的 .gitlint 配置的规则,来自动检查您的提交消息。在代码仓下的任意目录中执行如下命令即可:

$ gitlint install-hook

配置提交模板

  1. 在本地任意位置创建一个模板文件 template.txt , 内容填写你想要的模板。以下内容可以作为一个示例模板(记得替换下your-name和your-email为你自己的名字和邮箱):

area: subject

*why
*what
*how

Signed-off-by: your-name <your-email>
  1. 执行如下命令之一,选择进行全局配置或者当前代码仓配置,配置git的提交模板:

$ git config --local commit.template /path/to/template.txt   //需在对应代码仓的文件夹下执行,只为该代码仓提交时配置模板
$ git config --global commit.template /path/to/template.txt  //全局配置该模板