何を見つけたか
収束レビュー(cross-review)で、agy がレビューを Pull Request へ投稿し、結果ファイルへ
review_id を書いたが、レビューの URL(review_url)を書かなかった。進行側は URL が
無いことを「投稿が届いていない」と読み、結果なし(not_posted)として扱った。起動し直した
2 回目も同じ形になり、判定は中断(JUDGE_RC=1・final=error)で終わった。
| 何を見たか |
値 |
agy の結果ファイルの判定と review_id |
APPROVE / 5277384565(2 回目) |
結果ファイルの review_url |
無い |
| GitHub 上の agy のレビュー |
2 件とも届いている(5277319879 / 5277384565、どちらも COMMENTED) |
| 進行側の出力 |
agy: 投稿されたレビューを確認できません (review_url=None) |
実在するレビューの URL を結果ファイルへ補って取り込み直すと、APPROVE として取り込まれ収束した。
どこで見つけたか
なぜこの変更の範囲外なのか
PR #800 が変えるのは構造改善(cross-refactoring)の参加者の決め方で、収束レビューの結果の取り込みは受け入れ条件に含まれない。実行したのも develop の版である。
直さないと何が起きるか
agy が URL を書かない形を取るたびに、届いている承認が結果なしになり、起動し直しで所要が倍になったうえで中断する。PR #547 の形(#548)と同じく、届いている結果で収束が止まる。
由来
PR #800
何を見つけたか
収束レビュー(cross-review)で、agy がレビューを Pull Request へ投稿し、結果ファイルへ
review_idを書いたが、レビューの URL(review_url)を書かなかった。進行側は URL が無いことを「投稿が届いていない」と読み、結果なし(
not_posted)として扱った。起動し直した2 回目も同じ形になり、判定は中断(
JUDGE_RC=1・final=error)で終わった。review_idAPPROVE/5277384565(2 回目)review_url5277319879/5277384565、どちらもCOMMENTED)agy: 投稿されたレビューを確認できません (review_url=None)実在するレビューの URL を結果ファイルへ補って取り込み直すと、
APPROVEとして取り込まれ収束した。どこで見つけたか
plugins/ndf/skills/cross-review/scripts/state.pyの_verify_review_arrival)が URL だけで存在を照会し、URL が無いと「届いていない」と判定するなぜこの変更の範囲外なのか
PR #800 が変えるのは構造改善(cross-refactoring)の参加者の決め方で、収束レビューの結果の取り込みは受け入れ条件に含まれない。実行したのも
developの版である。直さないと何が起きるか
agy が URL を書かない形を取るたびに、届いている承認が結果なしになり、起動し直しで所要が倍になったうえで中断する。PR #547 の形(#548)と同じく、届いている結果で収束が止まる。
由来
PR #800