Merge requests workflow

• • Keep it simple
• Contribution acceptance criteria
• Dependencies

Merge requests workflow

We welcome merge requests from everyone, with fixes and improvements to GitLab code, tests, and documentation. The issues that are specifically suitable for community contributions are listed with the label, but you are free to contribute to any issue you want.

请注意，如果在任何时候都为当前里程碑标记了一个问题，即使您正在处理它，则 GitLab Inc.团队成员可能会接管合并请求，以确保工作在发布日期之前完成.

如果要添加未标记的新功能，最好首先创建一个问题（如果还没有一个问题）并发表评论，要求将其标记为” Accepting Merge Requests . 如果还可以更改用户界面，请提供建议功能的屏幕截图或线框.

合并请求应提交到 GitLab.com 上的相应项目，例如GitLab ， ， Omnibus GitLab等.

如果您不熟悉 GitLab 开发（或一般而言的 Web 开发），请参阅” 部分，以开始解决一些潜在的简单问题.

如果发现问题，请提交包含修复或改进的合并请求（如果可以），并包括测试. 如果您不知道如何解决该问题，但是可以编写暴露该问题的测试，我们也将接受. 通常，包含回归测试的错误修复将快速合并，而没有适当测试的新功能可能会更慢地接收反馈. 进行合并请求的工作流程如下：

1. 叉项目插入 GitLab.com 您的个人命名空间（或一组）.
2. 在 fork 中创建一个功能分支（不要使用master ）.
3. 编写和代码.
4. Generate a changelog entry with bin/changelog
5. 如果要编写文档，请确保遵循 .
6. 遵循提交消息准则 .
7. 如果您有多个提交，请通过将它们组合为几个逻辑组织的提交，但是如果您正在共享分支上，则不要更改提交历史记录.
8. 将提交推送到 fork 中的工作分支.
9. 向主 GitLab 项目中的master分支提交合并请求（MR）.
   1. 您的合并请求至少需要 1 个批准，但是根据您的更改，您可能需要其他批准. 请参阅批准准则 .
   2. 您不必选择任何特定的批准人，但是如果您确实希望特定的人批准您的合并请求，则可以选择.
10. MR 标题应描述您要进行的更改.
11. MR 说明应说明您进行更改的原因.
    1. 如果您要提供代码，请根据”描述”字段中已经提供的默认模板填写描述.
    2. 如果您要提供文档，请从”选择模板”菜单中选择” “，然后根据模板填写说明.
12. 如果允许，请设置相关的里程碑和 .
13. UI 更改应使用 GitLab 设计系统” 睡衣”中的可用组件. MR 必须包含” 之前和之后”屏幕截图.
14. 如果 MR 更改了 CSS 类，请包括受影响页面的列表，可以通过运行grep css-class ./app -R来找到.
15. 如果您的 MR 触摸了执行 Shell 命令，读取或打开文件或处理磁盘上文件路径的代码，请确保它符合
16. 如果您的代码在磁盘上创建了新文件，请阅读共享文件准则 .
17. 如果您的合并请求添加了一个或多个迁移，请确保在审阅 MR 之前对新数据库执行所有迁移. 如果检查导致 MR 发生较大变化，请在检查完成后再次执行迁移.
18. 为更复杂的迁移编写测试.
19. 合并请求必须遵守 .
20. 对于使用 Capybara 的测试，请阅读如何编写可靠的异步集成测试 .
21. 如果从源安装 GitLab 时，合并请求中引入的更改需要其他步骤， doc/install/installation.md在同一合并请求中将它们添加到中.
22. 如果您的合并请求引入了从源代码升级 GitLab 时需要其他步骤的更改，请在同一合并请求中将它们添加到doc/update/upgrading_from_source.md中. 如果这些说明是特定于版本的，请将它们添加到”特定于版本的升级说明”部分.
23. 阅读并遵守 .
24. 阅读并关注审核您的合并请求 .

如果您想对合并请求提供快速反馈，请随时提及或合并请求指导者之一 . 在审查您的代码以及审查合并请求时，请牢记 . 并且，如果您的代码也对数据库进行了更改或进行了昂贵的查询，请查看数据库复审指南 .

进行较小的迭代. 请保持单个 MR 中的更改量尽可能小 . 如果您想提供较大的功能，请仔细考虑 . 您可以将功能分为两个较小的 MR 吗？ 您只能提交后端/ API 代码吗？ 您可以从一个非常简单的 UI 开始吗？ 您可以只做一部分重构吗？

小型 MR 更易于查看，从而导致更高的代码质量，对于 GitLab 而言，这比具有最少的提交日志更为重要. MR 越小，合并的可能性就越大. 之后，您可以发送更多 MR，以增强和扩展功能. Kubernetes 团队的《 如何获得更快的 PR 评论》文档也对此有一些建议.

Commit messages guidelines

编写提交消息时，请遵循以下准则：

• 提交主题必须包含至少 3 个字.
• 提交主题不得超过 72 个字符.
• 提交主题必须以大写字母开头.
• 提交主题不得以句号结尾.
• 提交主题和正文必须用空白行分隔.
• 提交正文每行不得包含超过 72 个字符.
• 在至少 3 个文件中更改 30 行或更多行的提交必须在提交正文中描述这些更改.
• 提交主题或正文不得包含表情符号.
• 使用问题并合并请求的完整 URL 而不是简短参考，因为它们在 GitLab 之外显示为纯文本.
• 合并请求不得包含超过 10 条提交消息.

可以在您的计算机上使用的体现以上内容的示例提交消息模板（有关如何应用 template 的指南）：

为确保您的合并请求能够获得批准，请确保其符合以下捐款接受标准：

1. 更改尽可能小.
2. 包括适当的测试并使所有测试通过（除非它包含暴露现有代码中的错误的测试）. 每个新类都应具有相应的单元测试，即使该类是在较高级别上进行的，例如功能测试.
   • 如果失败的 CI 构建似乎与您的贡献无关，则可以尝试重新启动失败的 CI 作业，从主服务器重新部署以引入可以解决该失败的更新，或者如果尚未解决，请请开发人员来帮助您修复测试.
3. MR 最初包含一些按逻辑组织的提交.
4. 更改可以合并而不会出现问题. 否则，如果您是功能分支上的唯一成员，则应重新设置基准，否则合并master .
5. 仅解决了一个特定问题或实现了一个特定功能. 不要结合东西； 针对每个问题或功能发送单独的合并请求.
6. 迁移应仅做一件事（例如，创建表，将数据移动到新表或删除旧表），以帮助重试失败.
7. 不要添加配置选项或设置选项，因为它们会使进行和测试将来的更改复杂化.
8. 更改不会降低性能：
   • 避免重复轮询需要大量开销的端点.
   • 通过 SQL 日志或检查 N + 1 个查询.
   • 避免重复访问文件系统.
   • 如果需要支持实时功能，可将轮询与 ETag 缓存一起使用.
9. 如果合并请求添加了任何新库（宝石，JavaScript 库等），则它们应符合我们的 . 如果” license-finder”测试失败，并且Dependencies that need approval错误请参阅那些说明. 另外，使审阅者了解新库并解释为什么需要它.
10. 合并请求符合下面的 GitLab 对 done的 .

如果您为 GitLab 做贡献，请知道更改不仅涉及代码. 我们使用以下完成的定义 . 在您确定满足所有这些要求之前，您的贡献不会完成 .

1. 清晰的说明，说明捐款的相关性.
2. 工作并清理需要注释的代码.
3. 所有通过 CI 服务器的 .
4. 测试涵盖了回归和错误，可降低问题再次发生的风险.
5. 遵循性能准则 .
6. 遵循 .
7. 记录在/doc目录中.
8. 如有必要， .
9. 由相关（UX / FE / BE /技术写作）审稿人审阅，并解决所有问题.
10. 由项目维护者合并.
11. 在基础结构问题跟踪器中创建问题，以在您的文稿更改默认设置或引入新设置（如果相关）时通知基础结构部门.
12. 部署贡献后，确认已在或 GitLab.com 上工作.
13. 添加到发布中 （如果相关）.
14. 添加到 （如果相关）.
15. 如果需要， 可以添加 . 如有任何疑问，请联系质量团队 .

如果您在 GitLab 中添加了一个依赖项（例如操作系统软件包），请考虑更新以下内容，并在合并请求中注意每个依赖项的适用性：

1. 请注意的附加内容（如果尚不存在，请创建一个）.
2. The upgrade guide.
3. The .
4. The GitLab Development Kit.
5. The .
6. The Omnibus package creator.

我们会在工程上花费一些时间来解决小问题（有或没有问题），这些小问题是逐步改进的，例如：

1. 未经优先考虑的错误修复（例如 ）
2. Rubocop 或代码质量改进