如何提交代码和文档

如何提交代码和文档

前提条件

点击 Sign in with Github to agree 按钮签署 CLA 协议。

什么是 CLA？

Step 1: 通过 GitHub Fork

访问 https://github.com/vesoft-inc/nebula
点击右上角 Fork 按钮创建远程分支

Step 2: 将分支克隆到本地

定义本地工作目录：

# 定义工作目录
working_dir=$HOME/Workspace

将 user 设置为 GitHub 账户名：

user={GitHub账户名}

clone 代码：

mkdir -p $working_dir
cd $working_dir
git clone git@github.com:$user/nebula.git
# 或: git clone https://github.com/$user/nebula.git
cd $working_dir/nebula
git remote add upstream git@github.com:vesoft-inc/nebula.git
# 或: git remote add upstream https://github.com/vesoft-inc/nebula.git
# 由于没有写访问权限，请勿推送至上游主分支
git remote set-url --push upstream no_push
# 确认远程分支有效：
# 正确的格式为：
# origin    git@github.com:$(user)/nebula.git (fetch)
# origin    git@github.com:$(user)/nebula.git (push)
# upstream  https://github.com/vesoft-inc/nebula (fetch)
# upstream  no_push (push)
git remote -v

定义预提交 hook

请将 Nebula Graph 预提交挂钩链接到 .git 目录。

此挂钩检查提交格式，构建，文档生成等。

cd $working_dir/nebula/.git/hooks
ln -s ../../cpplint/bin/pre-commit.sh .

有时，预提交挂钩不能执行，在这种情况下，需要手动执行。

cd $working_dir/nebula/.git/hooks
chmod +x pre-commit

Step 3: 分支

更新本地主分支：

cd $working_dir/nebula
git fetch upstream
git checkout master
git rebase upstream/master

从主分支创建并切换分支：

git checkout -b myfeature

注意由于一个 PR 通常包含多个 commit，在合并至 master 时容易被挤压 ( squash )，因此建议创建一个独立的分支进行更改。合并后，这个分支已无用处，因此可以使用上述 rebase 命令将本地 master 与 upstream 同步。此外，如果直接将 commit 提交至 master，则需要 hard reset 主分支，例如：

git fetch upstream
git checkout master
git reset --hard upstream/master
git push --force origin master

Step 4: 开发

代码风格

Nebula Graph 采用 cpplint 来确保其代码符合 Google 的代码风格指南。此检查器将在提交代码之前实现。

添加单元测试

请为你的新功能或 bugfix 添加单元测试。在修改的模块代码目录下面有个 test 目录，可以在里面添加单元测试，然后编译运行单元测试，提交的代码必须确保所有单元测试顺利通过。

请开启单元测试进行源码编译

详情请参考源码编译。

请确保已设置 -DENABLE_TESTING = ON 启用了单元测试的构建。

运行所有单元测试

在 nebula 根目录运行以下命令：

cd nebula/build
ctest -j$(nproc)

验证

替换二进制文件

编译好的三个服务的二进制文件在 nebula/build/src/daemon/_build/ 目录下面，编译好的 console 在 nebula/build/src/console/_build 目录下面。可以把二进制替换到安装目录 bin 下面，重启服务并做验证。

Step 5: 保持分支同步

# 当处于 myfeature 分支时：
git fetch upstream
git rebase upstream/master

在其他协作者将 PR 合并到基础分支之后，您需要更新 head 分支。

Step 6: Commit

提交代码更改

git commit

Step 7: Push

代码更改完成或需要备份代码时，将本地仓库创建的分支 push 到 GitHub 端的远程仓库：

git push origin myfeature

Step 8: 创建 pull request

点击此处访问 fork 仓库https://github.com/$user/nebula (替换此处的 $user 用户名)。
点击 myfeature 分支旁的 Compare & pull request 按钮。

Step 9: 代码审查

公开的 pull request 至少需要两人审查，代码审查包括查找 bug，审查代码风格等。

如何贡献