返回介绍

参与贡献

发布于 2021-04-06 06:22:52 字数 2825 浏览 1118 评论 0 收藏 0

可以先去 发展路线 内了解下开发任务及未来规划。

准备工作

贡献代码前需要先了解 git 工具的使用和 GitHub 网站的使用。

GitHub 贡献代码流程

提交issue

不论您是修复 SOFAArk 的 bug 还是新增 SOFAArk 的功能,在您提交代码之前,在 SOFAArk 的 GitHub 地址上提交一个 issue,描述您要修复的问题或者要增加的功能。这么做有几个好处:

  • 不会与其它开发者或是他们对这个项目的计划发生冲突,产生重复工作.
  • SOFAArk 的维护人员会对您提的bug或者新增功能进行相关讨论,确定该修改是不是必要,有没有提升的空间或更好的办法。
  • 在达成一致后再开发,并提交代码,减少双方沟通成本,也减少pull request被拒绝的情况。

获取源码

要修改或新增功能,在提 issue 后,点击左上角的fork按钮,复制一份 SOFAArk 主干代码到您的代码仓库。

拉分支

SOFAArk 所有修改都在分支上进行,修改完后提交 pull request, 在 Code Review 后由项目维护人员 Merge 到主干。
因此,在获取源码步骤介绍后,您需要:

  • 下载代码到本地,这一步您可以选择git/https方式.

    git clone https://github.com/您的账号名/sofa-ark.git
    
  • 拉分支准备修改代码

    git branch add_xxx_feature
    

执行完上述命令后,您的代码仓库就切换到相应分支了。执行如下命令可以看到您当前分支:

  git branch -a

如果您想切换回主干,执行下面命令:

  git checkout -b master

如果您想切换回分支,执行下面命令:

  git checkout -b "branchName"

修改代码提交到本地

拉完分支后,就可以修改代码了。

修改代码注意事项

  • 代码风格保持一致

SOFAArk 通过 Maven 插件来保持代码格式一致.在提交代码前,务必本地执行

  mvn clean compile
  • 补充单元测试代码
  • 新有修改应该通过已有的单元测试.
  • 应该提供新的单元测试来证明以前的代码存在 bug,而新的代码已经解决了这些 bug

您可以用如下命令运行所有测试

  mvn clean test

也可以通过IDE来辅助运行。

其它注意事项

  • 请保持您编辑的代码的原有风格,尤其是空格换行等.
  • 对于无用的注释,请直接删除
  • 对逻辑和功能不容易被理解的地方添加注释。
  • 及时更新文档

修改完代码后,执行如下命令提交所有修改到本地:

  git commit -am '添加xx功能'

提交代码到远程仓库

在代码提交到本地后,就是与远程仓库同步代码了。执行如下命令提交本地修改到 github 上:

git push origin "branchname"

如果前面您是通过 fork 来做的,那么这里的 origin 是 push 到您的代码仓库,而不是 SOFAArk 的代码仓库.

提交合并代码到主干的请求

在的代码提交到 GitHub 后,您就可以发送请求来把您改好的代码合入 SOFAArk 主干代码了。此时您需要进入您的 GitHub 上的对应仓库,按右上角的 pull request按钮。选择目标分支,一般就是 master,系统会通知 SOFAArk 的人员, SOFAArk 人员会 Review 您的代码,符合要求后就会合入主干,成为 SOFAArk 的一部分。

代码 Review

在您提交代码后,您的代码会被指派给维护人员 Review,请耐心等待。如果在数天后,仍然没有人对您的提交给予任何回复,可以在 PR 下面留言,并 @ 对应的人员.

对于代码 Review 的意见会直接备注到到对应 PR 或者 Issue。如果觉得建议是合理的,也请您把这些建议更新到您的补丁中。

合并代码到主干

在代码 Review 通过后,就由 SOFAArk 维护人员操作合入主干了。这一步不用参与,代码合并之后,您会收到合并成功的提示。

如果你对这篇内容有疑问,欢迎到本站社区发帖提问 参与讨论,获取更多帮助,或者扫码二维码加入 Web 技术交流群。

扫码二维码加入Web技术交流群

发布评论

需要 登录 才能够评论, 你可以免费 注册 一个本站的账号。
列表为空,暂无数据
    我们使用 Cookies 和其他技术来定制您的体验包括您的登录状态等。通过阅读我们的 隐私政策 了解更多相关信息。 单击 接受 或继续使用网站,即表示您同意使用 Cookies 和您的相关数据。
    原文