contribute_to_paddle_cn.md 6.0 KB
Newer Older
L
update  
livc 已提交
1
# 如何贡献代码
L
livc 已提交
2

L
update  
livc 已提交
3
我们真诚地感谢您的贡献,欢迎通过 GitHub 的 fork 和 pull request 流程来提交代码。
L
livc 已提交
4

L
update  
livc 已提交
5 6 7 8 9
## 代码要求
- 你的代码必须完全遵守 [doxygen](http://www.stack.nl/~dimitri/doxygen/) 的样式。
- 确保编译器选项 WITH\_STYLE\_CHECK 已打开,并且编译能通过代码样式检查。
- 所有代码必须具有单元测试。
- 通过所有单元测试。
L
livc 已提交
10

L
update  
livc 已提交
11 12 13 14
以下教程将指导您提交代码。
## [Fork](https://help.github.com/articles/fork-a-repo/)

跳转到[PaddlePaddle](https://github.com/PaddlePaddle/Paddle) GitHub首页,然后单击 `Fork` 按钮,生成自己目录下的仓库,比如 <https://github.com/USERNAME/Paddle>
L
livc 已提交
15 16 17

## 克隆(Clone)

L
update  
livc 已提交
18 19 20 21 22 23
将远程仓库 clone 到本地。

```bash
➜  git clone https://github.com/USERNAME/Paddle
cd Paddle
```
L
livc 已提交
24 25


L
livc 已提交
26 27
## 创建本地分支

L
update  
livc 已提交
28 29 30
Paddle 目前使用[Git流分支模型](http://nvie.com/posts/a-successful-git-branching-model/)进行开发,测试,发行和维护。**develop** 是主分支,其他用户分支是特征分支(feature branches)。

所有的 feature 和 bug fix 的开发工作都应该在一个新的分支上完成,一般从 `develop` 分支上创建新分支。
L
livc 已提交
31 32 33 34 35 36 37

```bash
# (从当前分支)创建名为 MY_COOL_STUFF_BRANCH 的新分支
➜  git branch MY_COOL_STUFF_BRANCH

# 切换到这个分支上
➜  git checkout MY_COOL_STUFF_BRANCH 
L
livc 已提交
38 39
```

L
livc 已提交
40
也可以通过 `git checkout -b` 一次性创建并切换分支。
L
livc 已提交
41

L
livc 已提交
42 43
```bash
➜  git checkout -b MY_COOL_STUFF_BRANCH
L
livc 已提交
44 45
```

L
livc 已提交
46 47 48
值得注意的是,在 checkout 之前,需要保持当前分支目录 clean,否则会把 untracked 的文件也带到新分支上,这可以通过 `git status` 查看。

## 开始开发
L
livc 已提交
49

L
livc 已提交
50
在本例中,我删除了 README.md 中的一行,并创建了一个新文件。
L
livc 已提交
51

L
livc 已提交
52
通过 `git status` 查看当前状态,这会提示当前目录的一些变化,同时也可以通过 `git diff` 查看文件具体被修改的内容。
L
livc 已提交
53

L
livc 已提交
54 55 56 57 58 59
```bash
➜  git status
On branch test
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)
L
livc 已提交
60

L
livc 已提交
61
	modified:   README.md
L
livc 已提交
62

L
livc 已提交
63 64
Untracked files:
  (use "git add <file>..." to include in what will be committed)
L
livc 已提交
65

L
livc 已提交
66 67 68
	test

no changes added to commit (use "git add" and/or "git commit -a")
L
livc 已提交
69 70
```

L
livc 已提交
71 72 73
## 提交(commit)

接下来我们取消对 README.md 文件的改变,然后提交新添加的 test 文件。
L
livc 已提交
74

L
livc 已提交
75 76 77 78 79 80
```bash
➜  git checkout -- README.md
➜  git status
On branch test
Untracked files:
  (use "git add <file>..." to include in what will be committed)
L
livc 已提交
81

L
livc 已提交
82 83 84 85
	test

nothing added to commit but untracked files present (use "git add" to track)
➜  git add test
L
livc 已提交
86 87
```

L
livc 已提交
88
Paddle 使用 [pre-commit](http://pre-commit.com) 完成代码风格检查的自动化,它会在每次 commit 时自动检查代码是否符合规范,并检查一些基本事宜,因此我们首先安装并在当前目录运行它。
L
livc 已提交
89

L
livc 已提交
90 91 92
```bash
➜  pip install pre-commit
➜  pre-commit install
L
livc 已提交
93 94
```

L
livc 已提交
95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116
Git 每次提交代码,都需要写提交说明,这可以让其他人知道这次提交做了哪些改变,这可以通过`git commit -m` 完成。

```bash
➜  git commit -m "add test file"
CRLF end-lines remover...............................(no files to check)Skipped
yapf.................................................(no files to check)Skipped
Check for added large files..............................................Passed
Check for merge conflicts................................................Passed
Check for broken symlinks................................................Passed
Detect Private Key...................................(no files to check)Skipped
Fix End of Files.....................................(no files to check)Skipped
clang-formater.......................................(no files to check)Skipped
[MY_COOL_STUFF_BRANCH c703c041] add test file
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 233
```

## 保持本地仓库最新

在准备发起 Pull Request 之前,需要同步原仓库(<https://github.com/PaddlePaddle/Paddle>)最新的代码。

首先通过 `git remote` 查看当前远程仓库的名字。
L
livc 已提交
117

L
livc 已提交
118 119 120 121 122 123 124 125 126
```bash
➜  git remote
origin
➜  git remote -v
origin	https://github.com/USERNAME/Paddle (fetch)
origin	https://github.com/USERNAME/Paddle (push)
```

这里 origin 是我们 clone 的远程仓库的名字,也就是自己用户名下的 Paddle,接下来我们创建一个原始 Paddle 仓库的远程主机,命名为 upstream。
L
livc 已提交
127

L
livc 已提交
128 129 130 131 132
```bash
➜  git remote add upstream https://github.com/PaddlePaddle/Paddle
➜  git remote
origin
upstream
L
livc 已提交
133 134
```

L
livc 已提交
135
获取 upstream 的最新代码并更新当前分支。
L
livc 已提交
136

L
livc 已提交
137 138 139 140
```bash
➜  git fetch upstream
➜  git pull --rebase upstream develop
```
L
livc 已提交
141

L
livc 已提交
142
## Push 到远程仓库
L
livc 已提交
143

L
livc 已提交
144
将本地的修改推送到 GitHub 上,也就是 https://github.com/USERNAME/Paddle。
L
livc 已提交
145

L
livc 已提交
146 147 148
```bash
# 推送到远程仓库 origin 的 MY_COOL_STUFF_BRANCH 分支上
➜  git push origin MY_COOL_STUFF_BRANCH
L
livc 已提交
149 150
```

L
livc 已提交
151 152 153 154 155 156 157 158 159 160 161
## 建立 Issue 并完成 PR

建立一个 Issue 描述问题,记录它的编号。

在 Push 新分支后, https://github.com/USERNAME/Paddle 中会出现新分支提示,点击绿色按钮发起 PR。

![](https://ws1.sinaimg.cn/large/9cd77f2egy1fez1jq9mwdj21js04yq3m.jpg)

选择目标分支:

![](https://ws1.sinaimg.cn/large/9cd77f2egy1fez1ku4a5vj21am04st9l.jpg)
L
livc 已提交
162

L
livc 已提交
163
在 PR 的说明中,填写 `solve #Issue编号` 可以在这个 PR 被 merge 后,自动关闭对应的 Issue,具体请见 <https://help.github.com/articles/closing-issues-via-commit-messages/>
L
livc 已提交
164

L
livc 已提交
165
接下来等待 review,如果有需要修改的地方,参照上述步骤更新 origin 中的对应分支即可。
L
livc 已提交
166

L
livc 已提交
167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182
## 删除远程分支

在 PR 被 merge 进主仓库后,我们可以在 PR 的页面删除远程仓库的分支。

![](https://ws1.sinaimg.cn/large/9cd77f2egy1fez1pkqohzj217q05c0tk.jpg)

## 删除本地分支

最后,删除本地分支。

```bash
# 切换到 develop 分支
git checkout develop 

# 删除 MY_COOL_STUFF_BRANCH 分支
git branch -D MY_COOL_STUFF_BRANCH 
L
livc 已提交
183
```
L
livc 已提交
184 185

至此,我们就完成了一次代码贡献的过程。