快速掌握OpenHarmony社区贡献新流程 原创 精华

zhushangyuan_
发布于 2023-5-30 15:25
浏览
6收藏

已同步到 OH社区官网。欢迎扫描关注、阅读。

https://mp.weixin.qq.com/s/4IbrJTd32kcAyamsLgay6Q

快速掌握OpenHarmony社区贡献新流程-鸿蒙开发者社区

10分钟快速掌握OpenHarmony社区贡献新流程

为提升Issue和PR(Pull Request)的处理效率,OpenHarmony社区优化了Issue和PR处理流程,新支持了一系列交互命令和状态标签,用于明确处理阶段和当前处理责任人。社区CI Bot工具还提供了待办事项提醒能力,并能自动处理超期无效Issue和PR。流程交互更加友好,基于交互提示,可以获知下一步需要如何操作。本文会对社区贡献流程优化点进行介绍,不管您是社区贡献的老专家还是初涉开源社区的新手,都有必要花几分钟快速熟悉下OpenHarmony社区贡献流程的新优化点。流程也在持续优化中,如有变化,请以最新的为准。

需要注意的是,流程优化是为了辅助社区参与者提升处理效率,不会影响既有流程。如果不使用新支持的交互命令和状态标签,也可以使用既有流程正常处理Issue和PR。但是强烈推荐大家使用这些新优化功能,通过可以明确当前处理人,让Issue和PR更及时地得到响应处理。

1、新流程能解决什么问题

先回顾下社区Issue和PR处理时存在的问题痛点。经常关注社区的开发者会注意到,社区上未闭环的Issue和PR数量比较多,处理速度也比较缓慢。导致Issue和PR不能有效处理的原因主要是:社区Issue和PR未规范处理,比如Issue描述不规范,缺少详细描述、验证步骤等关键信息;PR门禁编译失败、格式检测失败、门禁检查失败,DCO失败、未参考检视意见修改等导致不能合入。社区Issue和PR处理流程也存在一些改进点,可以提升Issue和PR处理效率,比如当前缺少Issue责任人精准分配;缺少机制分配PR检视人,PR处理阶段不清晰;缺少处理超期时主动提醒功能等;对超期的Issue和PR不能自动处理等。

OpenHarmony社区为解决上述问题,对Issue和PR处理流程进行了优化,主要包含:

  • 标记状态标签明确处理阶段责任人

    通过标记状态标签识别处理责任阶段、明确处理人。如果Issue和PR提交不规范,会有标签显示当前处理责任人为提交人;如果提交的PR通过门禁测试,等待审核检视,当前处理责任人为committer;如果已分配检视人员,当前处理责任人就是代码检视人员,等等。

  • 主动提醒责任人处理待办事项

    CI Bot会发邮件每日提醒责任人处理名下的待办事项。是否接收邮件可以通过订阅配置。

  • 超期问题自动处理

    基于规则,对于一些可以自动处理的情况进行分析,进行自动化处理。比如,对于验收中的Issue,如果长期未确认,会自动进行关闭;对于门禁未通过等情况导致不符合合入标准的PR,超过一定时间,会自动关闭。

OpenHarmony社区通过这些流程优化来提升Issue和PR处理效率,下文会详细介绍流程的优化点和具体使用方法。

2、新流程介绍

以PR流程为例介绍新流程,如图1所示。我们按状态标签来分别讲解,也可以参考OpenHarmony社区Pull Request&Issue评论支持命令清单

2.1 Waiting_On_Author状态标签

PR提交人(社区贡献者)创建PR后,PR的标签为Waiting_On_Author,表示当前的责任人为PR提交人。CI Bot会提醒PR提交人及时处理该PR。如果PR提交人长时期未处理该PR,CI Bot会进行自动关闭。

如果PR提交人触发门禁构建,构建失败后,PR的标签依旧为Waiting_On_Author状态。如果检视人员或committer审核人员提交了检视意见,PR的标签会被标记为Waiting_On_Author状态。

2.2 Waiting_For_Review状态标签

当PR提交人评论start build(仓库配置门禁时使用该命令,如果未配置门禁,请使用code review命令),并且门禁构建成功后,PR的状态标签替代为Waiting_For_Review状态,表示表示当前的责任人为committer审核人员,需要由committer分配检视人员。CI Bot可以每日邮件定时提醒待办事项,催促分配检视人员。

2.3 Reviewing状态标签

Committer可以通过assign [@gitee_id1 @gitee_id2...]分配检视人员,可以通过空格分割来指定多个检视人员;如果命令中不指定gitee_id,committer安排自己为检视人员。分配检视人员后,PR的状态标签替代为Reviewing状态,表示当前的责任人为代码检视人员。

分配的检视人员需参与检视,给出检视意见,然后评论命令check comment提醒PR提交人处理;无检视意见时,评论命令lgtm,提醒committer审核处理。

2.4 Waiting_For_Merge状态标签

当所有检视人员均对分配的PR没有检视意见时,并在PR评论区评论命令lgtm后,CI Bot会提醒committer去审核该PR。此时,PR的状态标签变换为Waiting_For_Merge状态,表示当前的责任人为committer审核人员。

2.5 Merged 状态标签

对于Waiting_For_Merge状态标签的PR, 当committer审核通过后,PR的状态标签会自动变换为Merged状态,表示该PR成功合入。

图1 PR审核处理流程图

快速掌握OpenHarmony社区贡献新流程-鸿蒙开发者社区

3、流程处理实例讲解

本节以Pull Request处理流程来讲解,按PR的处理阶段分别来讲解。

3.1 提交修改Pull Request

当PR提交人提交一个PR后,CI Bot会自动评论,如下所示。根据提示,如果代码已经开发完毕,PR提交人在PR评论区评论start build来触发门禁。在触发门禁前状态标签为Waiting_On_Author,当前的处理责任人为PR提交人。

图2 新PR交互截图

快速掌握OpenHarmony社区贡献新流程-鸿蒙开发者社区

如果审核检视人员为PR提交检视意见后,PR的状态标签变为Waiting_On_Author,需要PR提交人优化修复提交的代码。当处理完毕,重新推送代码后,需要重新触发门禁。

注意:如果代码仓没有配置门禁,提示的内容稍有不同,需要评论的命令是code view

3.2 门禁构建

在门禁通过后,PR的状态标签会替代为Waiting_For_Review状态,如下图所示。此后,该PR的处理责任人为代码仓的Committer。Committer会负责分配检视人员或者审核该PR。

图3 门禁构建成功截图

快速掌握OpenHarmony社区贡献新流程-鸿蒙开发者社区

3.3 代码检视

当一个PR处于Waiting_For_Review状态时,Committer可以使用assign命令分配给检视人员进行检视,如下图所示。命令assign的具体用法,可以参考上一小节图片中的操作提示。当分配完毕检视人员,PR的状态标签会替代为Waiting_For_Review状态,当前的处理责任人为分配的检视人员。

图4分配检视人员截图

快速掌握OpenHarmony社区贡献新流程-鸿蒙开发者社区

如果检视人员发现检视的PR存在问题,提出检视意见后,需要评论下check comment通知PR提交人根据检视意见进行修改。PR的状态标签会替代为Waiting_On_Author状态,当前的处理责任人为PR提交人。

图5提醒处理检视意见截图

快速掌握OpenHarmony社区贡献新流程-鸿蒙开发者社区

如果PR不存在问题,检视人员认为可以合入,需要评论下lgtm即(look good to me)通知Committer审核合入该PR。PR的状态标签会替代为Waiting_For_Merge状态,当前的处理责任人为Committer。

图6提醒审核合入截图

快速掌握OpenHarmony社区贡献新流程-鸿蒙开发者社区

3.4 审核合入

当代码仓Committer认为PR满足合入要求,审核通过后,PR会合入,此时PR的状态标签会替代为Merged状态,PR成功合入。

图7审核合入截图

快速掌握OpenHarmony社区贡献新流程-鸿蒙开发者社区

4、CI Bot待办提醒

通过状态标签识别当前处理责任人后,就可以获取责任人的待办事项。通过记录打标签的开始时间,就可以计算当前处理阶段停留时间,从而可以发邮件提醒及时处理待办事项,并能自动化处理超期无效的Issue和PR。发邮件功能可以自行选择是否订阅。

4.1 每日待办提醒

如果您在社区有待办事项,社区会自动汇总并自动发邮件给您,提醒您及时处理。如果不想收到邮件,可以取消订阅。强烈推荐您保持订阅,可以及时收到在社区的待办事项。下图为收到的待办事项邮件示例。

图8 待办事项邮件截图

快速掌握OpenHarmony社区贡献新流程-鸿蒙开发者社区

4.2 自动超期处理

对于PR,审核检视人员需要及时响应处理;PR提交人也需要及时响应反馈的检视意见,如果长期未响应,不符合合入标准的PR,会在30天后被自动关闭。这样做是为了保持一个干净的社区贡献环境,也不用担心丢失代码,被关闭的PR也可以很容易被PR提交人重新打开。对于Issue,如果社区审核人员认为需要补充信息,非问题,以及需要issue验收确认时,如果issue提交人30天未响应,也会被自动关闭处理。在关闭之后,会提醒,请保持关注Issue和PR的变更信息。如下图所示:

图9 自动超期处理截图

快速掌握OpenHarmony社区贡献新流程-鸿蒙开发者社区

5、小结

本文对OpenHarmony社区贡献流程优化点进行了介绍,包含新支持的一系列交互命令和状态标签,以及CI Bot的每日待办事项邮件、自动超期处理等。如有疑问,欢迎随时来社区反馈。

©著作权归作者所有,如需转载,请注明出处,否则将追究法律责任
已于2023-6-25 15:02:05修改
9
收藏 6
回复
举报
9条回复
按时间正序
/
按时间倒序
红叶亦知秋
红叶亦知秋

很详细的PR处理流程讲解,学习了

2
回复
2023-5-30 16:44:53
离北况归
离北况归

哇哦,赞

1
回复
2023-5-31 18:33:59
诺舒华吃西瓜
诺舒华吃西瓜

共同为OpenHarmony出份力

1
回复
2023-5-31 18:39:00
麻辣香锅配馒头
麻辣香锅配馒头

没想到流程如此复杂,工作人员辛苦了

1
回复
2023-6-1 20:14:34
香菜太难吃了
香菜太难吃了

作者是在社区工作吗

1
回复
2023-6-4 22:02:50
zhushangyuan_
zhushangyuan_

已同步到 OH社区官网。欢迎扫描关注、阅读。

​https://mp.weixin.qq.com/s/4IbrJTd32kcAyamsLgay6Q​

image.pngimage.png

回复
2023-6-25 15:02:20
zhushangyuan_
zhushangyuan_ 回复了 香菜太难吃了
作者是在社区工作吗

曾经工作过

回复
2023-7-26 08:51:02
zhushangyuan_
zhushangyuan_ 回复了 麻辣香锅配馒头
没想到流程如此复杂,工作人员辛苦了

谢谢 共同努力

回复
2023-7-27 10:43:42
安苒anran0
安苒anran0

没想到流程如此复杂,工作人员辛苦了

回复
2023-8-7 12:50:37
回复
    相关推荐