拉取请求审查指南编辑

对 Beats 所做的每一项更改都必须符合高标准,虽然拉取请求的质量最终由作者负责,但 Beats 团队成员有责任在审查过程中进行验证。如果本文档不明确或不恰当,应以常识和共识为准。

代码风格编辑

每个人对代码风格都有自己的看法。为了避免在此问题上浪费时间,我们几乎完全依赖go fmthound来规范代码风格。如果这两个工具都没有报错,那么代码几乎肯定是没问题的。可能会有例外情况,但这种情况应该极其罕见。只有在最不寻常的情况下才应该推翻这些工具的判断。

不稳定测试编辑

随着软件项目的规模不断扩大,其测试用例的复杂性也在不断增加,随之而来的是一些测试变得不稳定的可能性。处理不稳定测试是每个人的责任。如果您注意到拉取请求构建失败的原因与推送的代码无关,请按照以下步骤操作

  1. 使用“不稳定测试”GitHub 问题模板创建一个问题,并附上“不稳定测试”标签。
  2. 创建一个 PR 来屏蔽或修复不稳定测试。
  3. 合并该 PR 并基于其进行变基,然后再继续执行原始 PR 的正常 PR 流程。