Skip to content

cross-review: agy が結果にレビューの URL を書かないと、届いている承認が結果なしになり中断する #802

Description

@takemi-ohama

何を見つけたか

収束レビュー(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

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions