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

L
livc 已提交
3
我们真诚地感谢您的贡献,欢迎通过 GitHub 的 fork 和 pull request 流程来提交代码。
L
livc 已提交
4 5 6
 
## 代码要求
- 你的代码必须完全遵守 [doxygen](http://www.stack.nl/~dimitri/doxygen/) 的样式。
L
livc 已提交
7
- 确保编译器选项 WITH\_STYLE\_CHECK 已打开,并且编译能通过代码样式检查。
L
livc 已提交
8 9 10 11 12 13 14
- 所有代码必须具有单元测试。
- 通过所有单元测试。

以下教程将指导您提交代码。
 
## [Fork](https://help.github.com/articles/fork-a-repo/)
 
L
livc 已提交
15
跳转到[PaddlePaddle](https://github.com/PaddlePaddle/Paddle) GitHub首页,然后单击 `Fork` 按钮。
L
livc 已提交
16 17 18

## 克隆(Clone)

L
livc 已提交
19
Paddle 目前使用[git流分支模型](http://nvie.com/posts/a-successful-git-branching-model/)进行开发,测试,发行和维护。
L
livc 已提交
20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44
**develop** 是主分支,其他用户分支是特征分支(feature branches)。

一旦你创建了一个fork,你可以使用你最喜欢的 git 客户端克隆你的仓库(repo)或只是直接在命令行输入:

```shell
# 克隆 fork 到本地
git clone --branch develop https://github.com/USERNAME/Paddle.git
```
如果你的仓库不包含 **develop** 分支,你只需自己创建它。

```shell
git clone https://github.com/USERNAME/Paddle.git Paddle
cd Paddle
git checkout -b develop  # 创建 develop 分支
git remote add upstream https://github.com/PaddlePaddle/Paddle.git  # 添加 upstream 到 baidu/Paddle
git pull upstream develop  # 更新 upstream
git submodule update --init --recursive
```

然后你可以通过做一个本地开发分支开始开发

```shell
git checkout -b MY_COOL_STUFF_BRANCH
```

L
livc 已提交
45 46 47 48 49 50 51
## 使用 `pre-commit` 钩子

Paddle 开发人员使用 [pre-commit](http://pre-commit.com/) 工具来管理git预提交钩子。 它可以帮助我们格式化源代码(cpp,python),在提交前检查一些基本事宜(每个文件只有一个 EOL 
,git 中不要添加大文件)。 `pre-commit`测试是 Travis-CI 中单元测试的一部分,不满足钩子
的 PR 不能提交代码到 Paddle。

你可以通过 `pip install pre-commit` 安装 [pre-commit](http://pre-commit.com/)
L
livc 已提交
52
目前 Paddle 使用 `clang-format` 来调整C/C++源代码格式。请确保 clang-format 版本在3.8以上。
L
livc 已提交
53 54 55 56 57

然后只需在 Paddle clone 目录中运行 `pre-commit install` 。当你
提交你的代码时,pre-commit 钩子会检查本地代码是否存在
不适合提交的东西,等等。

L
livc 已提交
58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89
## 提交(Commit)

提交你的代码:

```shell
# 显示工作树状态
git status
# 添加修改过的文件
git add xx
env EDITOR=vim git commit  # 你可以用 vim/nano/emacs 写下你的注释
```
提交信息的第一行是标题,其他行可以添加一些细节(如果有必要的话)。

## 保持 Fork 状态最新

在拉(pull)你的请求(request)之前,你应该从最新的 PaddlePaddle 同步代码。
为此,你需要首先添加远程(remote):

```shell
# 观察当前远程仓库配置
git remote -v
# 添加上游(upstream)仓库
git remote add upstream https://github.com/PaddlePaddle/Paddle.git
# 验证新的 upstream
git remote -v
```

用最新的 upstream 更新你的 fork:

```shell
git pull --rebase upstream develop
```
L
livc 已提交
90
如果本地没有提交,git 将简单地执行快进。但是,如果你一直在做一些改变(绝大多数情况下不应该),你可能要处理冲突。
L
livc 已提交
91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106

现在,你的本地主分支与上游修改的一致并是最新的。

## 推送(Push)到 GitHub

```shell
# 在 GitHub 上 push 你的仓库
git push -u origin MY_COOL_STUFF_BRANCH  # 创建远程分支 MY_COOL_STUFF_BRANCH 到 origin.
```

## 拉取请求(Pull Request)

转到 GitHub上 你 fork 的页面,选择你的开发分支并单击 **pull request 按钮**

## 使用最新版本更新你的 pull 请求

L
livc 已提交
107
在代码审查(code review)期间,由于 baidu/Paddle 中新的提交导致你的 pull 请求可能会失效。如果没有冲突,GitHub允许自动更新。 你可以点击 pull request 页面中的“更新分支(Update Branch)”按钮。 但是如果存在代码冲突,你需要手动进行更新。你需要在本地仓库执行如下命令:
L
livc 已提交
108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131

```shell
git checkout MY_COOL_STUFF_BRANCH
git pull upstream develop
# 你可能需要根据git提示解决冲突
# 创建并测试你的代码
git push origin MY_COOL_STUFF_BRANCH
```
现在你的 Pull Request 是最新的了。

## 修改你的 pull request

当根据审阅者的意见修改 pull 请求时,请使用“git commit”而不是“git commit --amend”来提交更改,以便审阅者可以看到新的请求和旧的请求之间的区别。

可能的命令是

```shell
git checkout MY_COOL_STUFF_BRANCH
git pull upstream develop   # 将本地更新到最新的代码库
# 可能会发生一些冲突
# 开始开发吧!
env EDITOR=vim git commit  # 添加修改日志
git push origin MY_COOL_STUFF_BRANCH
```