更新 security review 的逻辑
如 这个链接 所示,按道理此时是 mr 第一次被 approve,应该选取一个前端 reviewer 做安全审查,但是 triage 直接认为 Martin 完成了 review。 根据分析,虽然 triage 最近更新了区分前后端的 security review,但是有一部分逻辑仍需要更新:
- 当 mr approve 时,如果 approved by security reviewer,需要进一步判断 reviewer 是前端还是后端。
/cc @daveliu @songhuang
由 佳濠 高 编辑于