forked from vesoft-inc/nebula
-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[doc]pr-commit-msg (vesoft-inc#1993)
* pr-commit-msg * add examples Co-authored-by: dutor <[email protected]>
- Loading branch information
Showing
2 changed files
with
102 additions
and
0 deletions.
There are no files selected for viewing
51 changes: 51 additions & 0 deletions
51
docs/manual-CN/4.contributions/pull-request-commit-message-guidelines.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,51 @@ | ||
# PR 和 Commit Message 指南 | ||
|
||
本文档介绍的 PR 和 Commit Message 指南适用于所有 **Nebula Graph** 仓库。 所有提交至 `master` 分支的 commit 均必须遵循以下准则。 | ||
|
||
## Commit Message | ||
|
||
```bash | ||
<type>(<scope>): <subject> // scope is optional, subject is must | ||
|
||
<body> // optional | ||
|
||
<footer> // optional | ||
``` | ||
|
||
本指南参照 [AngularJS commit 规则](https://docs.google.com/document/d/1QrDFcIiPjSLDn3EL15IJygNPiHORgU1_OOAqWjiDU5Y/edit)。 | ||
|
||
- `<Type>` 描述 commit 类型。 | ||
- `<subject>` 是 commit 的简短描述。 | ||
- 如需添加详细信息,请添加空白行,然后以段落格式进行添加。 | ||
|
||
### Commit 类型 | ||
|
||
Type | Description | ||
---------| ---------------- | ||
Feature | 新功能 | ||
Fix | 修复 bug | ||
Doc | 文档 | ||
Style | 代码格式 | ||
Refactor | 代码重构 | ||
Test | 增加测试 | ||
Chore | 构建过程或辅助工具的变动 | ||
|
||
## Pull Request | ||
|
||
提交 PR 时,请在标题中包含所有更改的详细信息,并确保标题简洁。 | ||
|
||
PR 标题必需简明概括更改信息。 | ||
|
||
对于显而易见的简单更改,可不必添加描述。如果 PR 涉及复杂更改,请对更改进行概述。如果 PR 修复了相关 issue,请关联。 | ||
|
||
### Pull Request 模板 | ||
|
||
```bash | ||
What changes were proposed in this pull request? | ||
|
||
Why are the changes needed? | ||
|
||
Does this PR introduce any user-facing change? | ||
|
||
How was this patch tested? | ||
``` |
51 changes: 51 additions & 0 deletions
51
docs/manual-EN/4.contributions/pull-request-commit-message-guidelines.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,51 @@ | ||
# Pull Request and Commit Message Guidelines | ||
|
||
This document describes the commit message and Pull Request style applied to all **Nebula Graph** repositories. Every commit made *directly* to the `master` branch must follow the below guidelines. | ||
|
||
## Commit Message | ||
|
||
```bash | ||
<type>(<scope>): <subject> // scope is optional, subject is must | ||
|
||
<body> // optional | ||
|
||
<footer> // optional | ||
``` | ||
|
||
These rules are adopted from the [AngularJS commit convention](https://docs.google.com/document/d/1QrDFcIiPjSLDn3EL15IJygNPiHORgU1_OOAqWjiDU5Y/edit). | ||
|
||
- `<Type>` describes the kind of change that this commit is providing. | ||
- `<subject>` is a short description of the change. | ||
- If additional details are required, add a blank line, and then provide explanation and context in paragraph format. | ||
|
||
### Commit Types | ||
|
||
Type | Description | ||
---------| ---------------- | ||
Feature | New features | ||
Fix | Bug fix | ||
Doc | Documentation changes | ||
Style | Formatting, missing semi colons, ... | ||
Refactor | Code cleanup | ||
Test | New tests | ||
Chore | Maintain | ||
|
||
## Pull Request | ||
|
||
When you submit a Pull Request, please include enough details about all changes in the title but keep it concise. | ||
|
||
The title of a pull request must briefly describe the changes made. | ||
|
||
For very simple changes, you can leave the description blank as there’s no need to describe what will be obvious from looking at the diff. For more complex changes, give an overview of the changes. If the PR fixes an issue, make sure to include the GitHub issue-number in the description. | ||
|
||
### Pull Request Template | ||
|
||
```bash | ||
What changes were proposed in this pull request? | ||
|
||
Why are the changes needed? | ||
|
||
Does this PR introduce any user-facing change? | ||
|
||
How was this patch tested? | ||
``` |