ICP理解测试

[kavonm]

@maohuaming ICP理解测试，请提问

[maohuaming]

@kavonm 按照ICP流程，如果pr的review未通过，开发者根据review意见进行修改后，再次提交，committer再次组织review，那个这个pr就已经打上两次size标签了。是否允许？

[kavonm]

@maohuaming 再次提交的时候，committer需要根据评审过的工作量修改原来的size标签，而不是重新设置，所以不允许

[maohuaming]

@kavonm 多人review的情况下，工作量如何计算？

[kavonm]

@maohuaming 验收工作开始前，committer首先对PR标记size标签； 验收工作结束后，在全部参与reviewer结束评审关闭PR的时候，committer对参与验收工作的人员的工作量进行确认

[maohuaming]

@kavonm 多人review如何进行？

[kavonm]

@maohuaming

1. 多个reviewer是由committer来指定
2. reviewer中的人员创建icp issue
3. PR的assignee可以打上size label 后续流程与committer单独review相同，只不过在要等所有reviewer都标记approve才能允许assignee 进行merge

[maohuaming]

@kavonm ICP定义是哪几个单词的缩写？

[kavonm]

@maohuaming issue commit pr

[maohuaming]

@kavonm pull对应的分支应该如何命名?

[kavonm]

@maohuaming 按照 类型/开发者-目的 进行命名 可取的类型包括：bugfix(修 BUG) feature(功能特性开发) doc(文档工作)

[maohuaming]

@kavonm pull request包含的 commit 命名方式是什么？

[kavonm]

@maohuaming bug 修改类：fix #<issue 编号> , <描述信息> 开发工作类：for #<issue 编号> , <描述信息>