Timo Kaufmann
Timo Kaufmann
I just pushed a new version with a new [usage](https://github.com/timokau/marvin-mk2/blob/master/USAGE.md) model. This new model now makes it possible to assign reviewers and automate triage. For now I am the [only...
[Here](https://github.com/NixOS/nixpkgs/pull/102867) it changed the status to `awaiting_reviewer`. From my experience, ofBorg requested maintainers are often unresponsive. So until they actually start the review we should treat the PR as `needs_reviewer`.
This would make it easier to update (without me as the bottleneck and not requiring a deployment). It would also potentially make it more discoverable (could be integrated with `maintainers.nix`,...
https://github.com/timokau/marvin-mk2/issues/22 is only a one-time transition, but it should probably hold until the PR is moved out of draft. [This](https://github.com/NixOS/nixpkgs/pull/91662#issuecomment-710783772) reminder should not have happened (though it apparently wasn't unwelcome...
This is especially relevant after #76. When somebody leaves multiple comments in a pull request review, all of them (and the final review submission) can trigger the same -> awaiting_reviewer...
Currently it just runs whenever requested (when a relevant state was set) or after one day. It might also run whenever reviewers become available (maybe conditioned on the cached number...
It would be nice if marvin automatically handled merge conflicts, for example by asking the author to resolve them and set the status to `awaiting_changes`. References: - https://developer.github.com/v3/git/#checking-mergeability-of-pull-requests - https://developer.github.com/v3/pulls/...
Triage could automatically set to `awaiting_changes` and post an appropriate message when checks failed. Assigning a reviewer could be postponed until checks succeed. https://docs.github.com/en/github/searching-for-information-on-github/searching-issues-and-pull-requests#search-by-commit-status
GitHub doesn't always notify of those, for example I didn't receive a notification for the force-push in https://github.com/NixOS/nixpkgs/pull/85115. There is a tradeoff with noisiness and false-positives here.
Hi first of all, thanks for this awesome plugin. I have a vim instance open with it running all the time. Because of this, I would really like it auto-updating....