gh pr view 131 --json createdAt,reviews --jq '{createdAt, reviews:[.reviews[]|{state,submittedAt}]}'
gh pr list --state merged --limit 5 --json number,title,mergedAt
发布单 18:00 截止,17:05 才发起评审,三个审批人 2 分钟内全部通过。事后查这次改动引入了一个空指针,19:20 线上报 500。这类现象通常不是突然出现的,往前翻半天日志能看到苗头。
追下去是这么回事:开发把评审当成上线前的手续,不是开发过程中的环节。审批人被截止时间推着走,只看有没有填完。
「评审拖到上线前」改完先别急着全量,挑一台机器放半天,真有问题这段时间就能看出来。权限不要一口气开到最大,先加到够用再往上试,范围小才好收。
把复现步骤压到三条以内,交给同事也能跑出来,才算真的定位清楚。
处理上按这个顺序:规定评审必须在提测前完成 上线前只做回归确认;发布单里加一条 评审通过时间距上线不少于 4 小时;超过 300 行的改动要求提前一天发起;审批人看不到改动说明就退回。
确认没问题再收工,评审通过时间比发布单截止时间早 4 小时以上,审批记录里能看到逐条评论而不是一次性通过。
