Christopher Dignam

Hey @sindrig, thanks for the report. This is weird behavior. I'm not sure why for example this PR is marked as missing reviews: https://github.com/island-is/island.is/pull/5717 It seems like it was merging...

I'm going to try eliminating those incorrect "missing required reviews" status messages in #762

Yeah, I meant two internal workers in Kodiak (actually asyncio tasks) processing the same repo. Kodiak starts repo worker tasks with this code: https://github.com/chdsbd/kodiak/blob/0c6cda5169e2f311ed654877b11a15feba65ad05/bot/kodiak/queue.py#L519-L525 I'm honestly not sure yet how...

Looking in the logs, it seems like the issue of having two workers at the same time isn't happening anymore, but, it seems like I introduced a small bug via...

I've deployed #765 which should help with some of the unnecessary updates when changes are requested

@Thorbenl Is this for pull request 9739? It looks like `build_and_test` is the only required status check for the `develop` branch and it looks like `build_and_test` passed for that PR,...

Nevermind, I was looking at the wrong PR. This is PR 9715. It looks like the last commit on that PR had a failing check run for `build_and_test`. I'm not...

I haven't been able to replicate this issue an a test repository. I wonder if there was some inconsistency with the GitHub API. Are you able to reliably replicate this...

@Thorbenl I would try opening an issue with GitHub Support because it shouldn't be possible to merge a pull request with a failing status check.

I wonder if Kodiak evaluated PR 58 and deleted the branch because it's the same name as used by that PR (`clxmstaab-patch-2`). (edit: this shouldn't really happen because we do...