Pull Request 须知
- 前言
- 规范

Pull Request 须知

前言

Pull Request 本质上是一种软件的合作方式，是将涉及不同功能的代码，纳入主干的一种流程。这个过程中，可以进行讨论、审核和修改代码。

在 Pull Request 中尽量不讨论代码的实现方案，代码及其逻辑的大体实现方案应该尽量在 Issue 或者邮件列表中被讨论确定，在 Pull Request 中我们尽量只关注代码的格式以及代码规范等信息，从而避免实现方式的意见不同而导致 waste time。

规范

Pull Request 标题

标题格式：[Pull Request 类型-Issue 号][模块名] Pull Request 描述

其中Pull Request 类型和Issue 类型的对应关系如下：

Issue 类型	Pull Request 类型	样例（假设 Issue 号为 3333）
Feature	Feature	[Feature-3333][server] Implement xxx
Bug	Fix	[Fix-3333][server] Fix xxx
Improvement	Improvement	[Improvement-3333][alert] Improve the performance of xxx
Test	Test	[Test-3333][api] Add the e2e test of xxx
Sub-Task	Sub-Task 对应的父类型	[Feature-3333][server] Implement xxx

其中 Issue 号是指当前 Pull Request 对应要解决的 Issue 号，模块名同 Issue 的模块名。

Pull Request 分支名

分支名格式：Pull Request 类型-Issue 号，举例：Feature-3333。

Pull Request 内容

请参阅到 commit message 篇。

Pull Request Code Style

当你向 DolphinScheduler 提交 pull request 的时候 code-style 是你不得不考虑的问题。我们在 CI 中使用 Checkstyle 参考来保持代码风格的统一，它是一种帮助开发者编写遵循编码规范的 Java 代码开发工具。如果你的 pull request 没有通过 Checkstyle 的检测，那它将不会被合并到主库中。你可以在提交 pull request 前使用 Checkstyle 来检测或者格式化你的代码。如下的步骤将引领你配置并激活 Checkstyle

准备 Checkstyle 配置文件：你可以点击这里手动下载，但是我们更加推荐在 DolphinScheduler 代码库中找到它。当你将代码库克隆下来后，你可以在路径 style/checkstyle.xml 下找到配置文件
下载 Intellij IDEA Checkstyle 插件：通过关键字CheckStyle-IDEA或者通过这个页面安装均可。如果你不清楚如何安装Intellij IDEA插件，可以参考这个连接
配置并激活 Checkstyles 以及 Intellij IDEA 代码风格：当完成上面几步后，你就可以配置并激活他们了。你可以在路径Preferences -> Tool -> Checkstyle中找到 Checkstyle，请参照下图完成其配置

截止目前，Checkstyle 插件已经配置完成了，当有代码或者文件不符合风格时就会显示在 Checkstyle 中。但强烈建议同时配置 Intellij IDEA 的代码风格，完成配置后可以使用 Intellij IDEA 自动格式化功能。你可以在路径Preferences -> Editor -> Code Style -> Java找到配置，请参照下图完成其配置

在提交 pull request 前格式化你的代码：完成上面全部后，你可以使用快捷键Command + L(Mac用户) or Ctrl+L(Windows用户)在 Intellij IDEA 完成自动格式化。格式化代码的最佳时间是将你的修改提交到本地 git 版本库之前

Pull Request须知

Pull Request 须知

前言

规范

Pull Request 标题

Pull Request 分支名

Pull Request 内容

Pull Request Code Style

相关问题