遥测审查指南¶

审查更改的一般指南¶

这些是我们审查更改时遵循的一般原则。

对于任何更改，这些都是我们始终需要得到令人满意答案的基本问题。

除了上述基本考虑因素外，以下是一些有助于审查更改的其他详细考虑因素。

遵循我们的标准和最佳实践。
- Firefox 桌面版
  - Mozilla 编码风格
  - 工具包代码审查指南
- 移动版
  - Android/Kotlin 代码风格
  - iOS/Swift 代码风格
这是否会对性能产生重大影响？
- 不要延迟应用程序初始化（除非绝对必要）。
- 不要阻塞网络请求。
- 尽可能使较长的任务异步化。
这是否比预期更广泛地影响产品？
- 考虑我们所有的平台：Windows、Mac、Linux、Android。
- 考虑我们所有的产品：Firefox、Fenix、Glean。
这是否违反了常见的架构缺陷？
- 除非编写解析器，否则优先使用采用非字符串类型的 API。
健全性检查
- 此更改在发布版本中的行为如何？您是否已测试过？
- 此更改是否包含测试覆盖率？我们要求对每个新代码、更改和错误修复进行测试覆盖率。
这是否需要更新文档？
- 到树内文档？
- 到firefox-data-docs（存储库）？
- 到glean 文档？
后续操作
- 我们是否已提交验证错误？
- 所有 TODO 是否都已提交后续错误？
- 我们是否需要向我们的用户传达此信息？
  - fx-data-dev（主要的 Firefox 数据列表）
  - firefox-dev（Firefox 应用程序开发人员）
  - dev-platform（Gecko/平台开发人员）
  - mobile-firefox-dev（移动开发人员）
  - fx-team（Firefox 员工）
- 我们是否需要向其他群体传达此信息？
  - 数据工程、数据科学、数据管理员……？

此更改是否可能破坏上游管道作业？
- 此更改是否以未处理的方式更改了传出数据的格式？
  - 例如，通过添加、删除或更改非指标有效负载字段的类型。
- 这是否需要更新 ping 模式？
- 这是否会破坏为指标解析注册表文件的作业？（Scalars.yaml、metrics.yaml 等）
我们是否需要让其他人参与？
- 数据格式、ping 内容、ping 语义等的更改需要让数据工程师参与。
- 对任何正在使用的传出数据进行更改都需要让利益相关者参与（例如，数据科学家）。