Upgrade review-bot

[mutantcornholio]

Upgrading PAPI inside review-bot

• Does not require a CHANGELOG entry

[bkontur]

@mutantcornholio, will you include this paritytech/get-fellows-action#38 as part of this PR?

We definitely need to merge it because, for example, the Review-Trigger job on this PR has been running for over 5 hours: https://github.com/polkadot-fellows/runtimes/actions/runs/12825386773/job/35763245073?pr=549.

[github-actions]

Review required! Latest push from author must always be reviewed

[mutantcornholio]

@bkontur done! Need another approve though

[bkchr]

@mutantcornholio https://github.com/polkadot-fellows/runtimes/actions/runs/12833435706/job/35872070272?pr=549

The output is a little bit overwhelming :D

[bkchr]

Also why does it require another review or what is the current error?

[mutantcornholio]

Also why does it require another review or what is the current error?

So on PR update, this code requires another review from everyone, who previously approved the PR.
Which for this PR is

{
  "reviewers": [
    "bkontur",
    "bkchr",
    "xlc",
    "seadanda",
    "muharem"
  ]
}

cc/ @xlc @seadanda @muharem

[bkchr]

So on PR update, this code requires another review from everyone, who previously approved the PR.

But why from all? Wouldn't it be enough for one of them to re-review.

[mutantcornholio]

So on PR update, this code requires another review from everyone, who previously approved the PR.

But why from all? Wouldn't it be enough for one of them to re-review.

Yeah, doesn't make sense for me as well. I can try searching for paper trail, to see why is it like this, or we can just change it (separately from these changes).

[bkchr]

Yeah, doesn't make sense for me as well. I can try searching for paper trail, to see why is it like this, or we can just change it (separately from these changes).

Sounds like it got mixed up. In Polkadot-sdk we only require the approval of one person again.

[mutantcornholio]

@bkchr I've created an issue about this: #557