-
Notifications
You must be signed in to change notification settings - Fork 35
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Showing
1 changed file
with
7 additions
and
7 deletions.
There are no files selected for viewing
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 |
---|---|---|
|
@@ -33,7 +33,7 @@ | |
|
||
#### 1. 复刻仓库 | ||
|
||
当你第一次提交拉取请求时,先复刻仓库原代码库,这里以 DIOPI-IMPL 为例。点击 GitHub 页面右上角的 **Fork** 按钮,复刻后的代码库将会出现在你的 GitHub 个人主页下。 | ||
当你第一次提交拉取请求时,先复刻仓库原代码库,这里以 IMPL 为例。点击 GitHub 页面右上角的 **Fork** 按钮,复刻后的代码库将会出现在你的 GitHub 个人主页下。 | ||
<!-- | ||
(图片待修改) | ||
<img src="" width="1200"> | ||
|
@@ -47,16 +47,16 @@ git clone [email protected]:{username}/DIOPI.git | |
添加原代码库为上游代码库 | ||
|
||
```bash | ||
git remote add upstream [email protected]:OpenComputeLab/DIOPI | ||
git remote add upstream [email protected]:DeepLink-org/DIOPI | ||
``` | ||
|
||
检查 remote 是否添加成功,在终端输入 `git remote -v` | ||
|
||
```bash | ||
origin [email protected]:{username}/DIOPI.git (fetch) | ||
origin [email protected]:{username}/DIOPI.git (push) | ||
upstream [email protected]:OpenComputeLab/DIOPI (fetch) | ||
upstream [email protected]:OpenComputeLab/DIOPI (push) | ||
upstream [email protected]:DeepLink-org/DIOPI (fetch) | ||
upstream [email protected]:DeepLink-org/DIOPI (push) | ||
``` | ||
|
||
> 这里对 origin 和 upstream 进行一个简单的介绍,当我们使用 git clone 来克隆代码时,会默认创建一个 origin 的 remote,它指向我们克隆的代码库地址,而 upstream 则是我们自己添加的,用来指向原始代码库地址。当然如果你不喜欢他叫 upstream,也可以自己修改,比如叫 diopi。我们通常向 origin 提交代码(即 fork 下来的远程仓库),然后向 upstream 提交一个 pull request。如果提交的代码和最新的代码发生冲突,再从 upstream 拉取最新的代码,和本地分支解决冲突,再提交到 origin。 | ||
|
@@ -78,7 +78,7 @@ git pull upstream master | |
|
||
|
||
#### 3. 提交代码并在本地通过一致性测试 | ||
提交的代码需要通过一致性测试套件以保证实现的正确性,具体可以参考一致性测试套件的[README](https://github.com/OpenComputeLab/DIOPI/DIOPI-TEST)。 | ||
提交的代码需要通过一致性测试套件以保证实现的正确性,具体可以参考一致性测试套件的[README](https://github.com/DeepLink-org/DIOPI/diopi-test)。 | ||
|
||
|
||
#### 4. 推送代码到远程 | ||
|
@@ -130,7 +130,7 @@ git push -u origin {branch_name} | |
--> | ||
所有 reviewer 同意合入 PR 后,我们会尽快将 PR 合并到主分支。 | ||
|
||
(5) 当前只有DIOPI-IMPL文件夹支持开发者贡献代码,DIOPI-PROTO与DIOPI-TEST后续会逐步开放。 | ||
(5) 当前只有IMPL文件夹支持开发者贡献代码,PROTO与DIOPI-TEST后续会逐步开放。 | ||
|
||
#### 6. 解决冲突 | ||
|
||
|
@@ -169,4 +169,4 @@ git merge upstream/master | |
- 描述里介绍`拉取请求`的主要修改内容,结果,以及对其他部分的影响, 参考`拉取请求`模板 | ||
- 关联相关的`议题` (issue) 和其他`拉取请求` | ||
|
||
5. 如果引入了其他三方库,或借鉴了三方库的代码,请确认他们的许可证和 DIOPI License 兼容,并在借鉴的代码上补充 `This code is inspired from http://` | ||
5. 如果引入了其他三方库,或借鉴了三方库的代码,请确认他们的许可证和 DIOPI License 兼容,并在借鉴的代码上补充 `This code is inspired from http://` |