跳到主要内容

如何评估 Pull Request 的影响

判断一个 Pull Request 是否应该被标记为“高影响”是一项复杂的任务。为了实现这一点，我们需要评估和估算 Pull Request 中引入的更改对 WooCommerce 的影响，这通常是一项主观的任务，并且有时会不准确，因为这需要对 WooCommerce 产品细节、技术细节，甚至客户问题历史有深入的了解。

在本页面，我们将分享一些指南，以帮助您评估 Pull Request 的影响程度。

您应该将 Pull Request 标记为“高影响”的情况：​

  • 它为 WooCommerce 添加了新功能，除非该功能位于特性开关 (feature flag) 后面。
  • 它修改了关键功能（请参阅 关键流程列表）。
  • 它修复了一个高优先级 bug（这包括修复 Blocks 的 bug 以及核心版本的更新）。
  • 它包含安全修复。
  • 它更新了SQL 查询。
  • 它涉及任何 $_REQUEST 变量。
  • 任何类型的数据迁移/更新。
  • 更改了 WooCommerce 发送的邮件。
  • 更改了 WooCommerce 的钩子/操作/过滤器。
  • 更改了 REST API 端点。
  • 这是一个大型 PR（即，它在许多文件中添加了多个更改）。
  • 它包含国际化 (i18n) 更改（例如，woocommerce/i18n 目录下的任何文件被修改）。

您不应该将 Pull Request 标记为“高影响”的情况：​

  • 它仅更新了自动化测试，与 WooCommerce 发布包中不包含的基础设施相关的内容，或 monorepo 中未包含在发布包中的其他项目。
  • 它仅包含文档 (readme) 或变更日志 (changelog) 的更改。
  • 修复了一个低优先级的 bug，例如拼写错误等。
  • 不需要验证在多种环境类型中。
  • 定期计划的（非修复版本）核心版本的更新，适用于 Blocks 包（因为测试已经安排）。
  • 它是尚未完整发布的产品的一部分（即，它当前位于特性开关 (feature flag) 后面，处于进行中状态）。

我的 PR 是高影响的。接下来该怎么办？​

如果您的 PR 是高影响的，请务必使用 impact: high 标签，WooCommerce 核心团队将对此进行特殊的测试考虑。