|
| 1 | +# 如何贡献代码 |
| 2 | + |
| 3 | +我们真诚地感谢您的贡献,欢迎通过 GitHub 的 fork 和 pull request 流程来提交代码。 |
| 4 | + |
| 5 | +## 代码要求 |
| 6 | +- 你的代码必须完全遵守 [doxygen](http://www.stack.nl/~dimitri/doxygen/) 的样式。 |
| 7 | +- 确保编译器选项 WITH\_STYLE\_CHECK 已打开,并且编译能通过代码样式检查。 |
| 8 | +- 所有代码必须具有单元测试。 |
| 9 | +- 通过所有单元测试。 |
| 10 | + |
| 11 | +以下教程将指导您提交代码。 |
| 12 | + |
| 13 | +## [Fork](https://help.github.com/articles/fork-a-repo/) |
| 14 | + |
| 15 | +跳转到[PaddlePaddle](https://github.com/PaddlePaddle/Paddle) GitHub首页,然后单击 `Fork` 按钮。 |
| 16 | + |
| 17 | +## 克隆(Clone) |
| 18 | + |
| 19 | +Paddle 目前使用[git流分支模型](http://nvie.com/posts/a-successful-git-branching-model/)进行开发,测试,发行和维护。 |
| 20 | +**develop** 是主分支,其他用户分支是特征分支(feature branches)。 |
| 21 | + |
| 22 | +一旦你创建了一个fork,你可以使用你最喜欢的 git 客户端克隆你的仓库(repo)或只是直接在命令行输入: |
| 23 | + |
| 24 | +```shell |
| 25 | +# 克隆 fork 到本地 |
| 26 | +git clone --branch develop https://github.com/USERNAME/Paddle.git |
| 27 | +``` |
| 28 | +如果你的仓库不包含 **develop** 分支,你只需自己创建它。 |
| 29 | + |
| 30 | +```shell |
| 31 | +git clone https://github.com/USERNAME/Paddle.git Paddle |
| 32 | +cd Paddle |
| 33 | +git checkout -b develop # 创建 develop 分支 |
| 34 | +git remote add upstream https://github.com/PaddlePaddle/Paddle.git # 添加 upstream 到 baidu/Paddle |
| 35 | +git pull upstream develop # 更新 upstream |
| 36 | +git submodule update --init --recursive |
| 37 | +``` |
| 38 | + |
| 39 | +然后你可以通过做一个本地开发分支开始开发 |
| 40 | + |
| 41 | +```shell |
| 42 | +git checkout -b MY_COOL_STUFF_BRANCH |
| 43 | +``` |
| 44 | + |
| 45 | +## 使用 `pre-commit` 钩子 |
| 46 | + |
| 47 | +Paddle 开发人员使用 [pre-commit](http://pre-commit.com/) 工具来管理git预提交钩子。 它可以帮助我们格式化源代码(cpp,python),在提交前检查一些基本事宜(每个文件只有一个 EOL |
| 48 | +,git 中不要添加大文件)。 `pre-commit`测试是 Travis-CI 中单元测试的一部分,不满足钩子 |
| 49 | +的 PR 不能提交代码到 Paddle。 |
| 50 | + |
| 51 | +你可以通过 `pip install pre-commit` 安装 [pre-commit](http://pre-commit.com/), |
| 52 | +目前 Paddle 使用 `clang-format` 来调整C/C++源代码格式。请确保 clang-format 版本在3.8以上。 |
| 53 | + |
| 54 | +然后只需在 Paddle clone 目录中运行 `pre-commit install` 。当你 |
| 55 | +提交你的代码时,pre-commit 钩子会检查本地代码是否存在 |
| 56 | +不适合提交的东西,等等。 |
| 57 | + |
| 58 | +## 提交(Commit) |
| 59 | + |
| 60 | +提交你的代码: |
| 61 | + |
| 62 | +```shell |
| 63 | +# 显示工作树状态 |
| 64 | +git status |
| 65 | +# 添加修改过的文件 |
| 66 | +git add xx |
| 67 | +env EDITOR=vim git commit # 你可以用 vim/nano/emacs 写下你的注释 |
| 68 | +``` |
| 69 | +提交信息的第一行是标题,其他行可以添加一些细节(如果有必要的话)。 |
| 70 | + |
| 71 | +## 保持 Fork 状态最新 |
| 72 | + |
| 73 | +在拉(pull)你的请求(request)之前,你应该从最新的 PaddlePaddle 同步代码。 |
| 74 | +为此,你需要首先添加远程(remote): |
| 75 | + |
| 76 | +```shell |
| 77 | +# 观察当前远程仓库配置 |
| 78 | +git remote -v |
| 79 | +# 添加上游(upstream)仓库 |
| 80 | +git remote add upstream https://github.com/PaddlePaddle/Paddle.git |
| 81 | +# 验证新的 upstream |
| 82 | +git remote -v |
| 83 | +``` |
| 84 | + |
| 85 | +用最新的 upstream 更新你的 fork: |
| 86 | + |
| 87 | +```shell |
| 88 | +git pull --rebase upstream develop |
| 89 | +``` |
| 90 | +如果本地没有提交,git 将简单地执行快进。但是,如果你一直在做一些改变(绝大多数情况下不应该),你可能要处理冲突。 |
| 91 | + |
| 92 | +现在,你的本地主分支与上游修改的一致并是最新的。 |
| 93 | + |
| 94 | +## 推送(Push)到 GitHub |
| 95 | + |
| 96 | +```shell |
| 97 | +# 在 GitHub 上 push 你的仓库 |
| 98 | +git push -u origin MY_COOL_STUFF_BRANCH # 创建远程分支 MY_COOL_STUFF_BRANCH 到 origin. |
| 99 | +``` |
| 100 | + |
| 101 | +## 拉取请求(Pull Request) |
| 102 | + |
| 103 | +转到 GitHub上 你 fork 的页面,选择你的开发分支并单击 **pull request 按钮**。 |
| 104 | + |
| 105 | +## 使用最新版本更新你的 pull 请求 |
| 106 | + |
| 107 | +在代码审查(code review)期间,由于 baidu/Paddle 中新的提交导致你的 pull 请求可能会失效。如果没有冲突,GitHub允许自动更新。 你可以点击 pull request 页面中的“更新分支(Update Branch)”按钮。 但是如果存在代码冲突,你需要手动进行更新。你需要在本地仓库执行如下命令: |
| 108 | + |
| 109 | +```shell |
| 110 | +git checkout MY_COOL_STUFF_BRANCH |
| 111 | +git pull upstream develop |
| 112 | +# 你可能需要根据git提示解决冲突 |
| 113 | +# 创建并测试你的代码 |
| 114 | +git push origin MY_COOL_STUFF_BRANCH |
| 115 | +``` |
| 116 | +现在你的 Pull Request 是最新的了。 |
| 117 | + |
| 118 | +## 修改你的 pull request |
| 119 | + |
| 120 | +当根据审阅者的意见修改 pull 请求时,请使用“git commit”而不是“git commit --amend”来提交更改,以便审阅者可以看到新的请求和旧的请求之间的区别。 |
| 121 | + |
| 122 | +可能的命令是 |
| 123 | + |
| 124 | +```shell |
| 125 | +git checkout MY_COOL_STUFF_BRANCH |
| 126 | +git pull upstream develop # 将本地更新到最新的代码库 |
| 127 | +# 可能会发生一些冲突 |
| 128 | +# 开始开发吧! |
| 129 | +env EDITOR=vim git commit # 添加修改日志 |
| 130 | +git push origin MY_COOL_STUFF_BRANCH |
| 131 | +``` |
0 commit comments