Timo Kaufmann
Timo Kaufmann
I just discovered in marvins logs that I had been re-requested for #91570. I didn't get an email. I have implemented the reminder by now, which somewhat alleviates the situation....
I think a good way to handle the "I don't feel qualified" case is to try to identify somebody who has more experience in that area and request a review...
Yes, I agree. The switch would also have [other benefits](https://github.com/timokau/marvin-mk2/issues/34#issuecomment-714386315). I'm not sure if we should also change the label names in that case.
Yes, its both. Being listed as a reviewer will actively request your reviews on `needs_reviewer` PRs. It will only do so if you don't already have a lot of activity...
> You can add me as a reviewer. What rate-limit would you like? As described above, the bot will count how many active (within `d` days) PRs you are currently...
> Perhaps might be a good idea to limit reviewers based on the attributes of PRs Maybe, at some point in the future. I'm not entirely sure its desirable though....
We now have 6 reviewers, great! > I'm a bit internally conflicted as my semester is about to finish and my tests are up coming. I understand, I'm in a...
> Is there any mechanism to find a different "PR shepherd" if the first one isn't fit for the job (e.g. inappropriate responses of some kind)? You can manually set...
In case anybody was missing any bot actions: A bug crashed the triage runner. I forgot that you can only request reviews from people who have been "explicitly granted read...
By the way, we are currently at capacity. So if a PR isn't immediately being assigned a reviewer, that is probably not a bug: ``` Jul 09 16:31:38 marvin-mk2: Running...