marvin-mk2
marvin-mk2 copied to clipboard
timokau
Discontinued! See https://github.com/timokau/marvin-mk2/issues/34#issuecomment-1100656280 (Previously: "Making sure your PR gets a review and your reviews don't get lost.")
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...
See this PR: https://github.com/NixOS/nixpkgs/pull/91662 The bot has added and removed the same labels a few times. Maybe that was just for debugging purposes, or maybe it's a bug.
Hey! From my experience with it I'm thinking Marvin is really great already, and could probably already be released on some percentage of all incoming github PRs (just changing the...
Hey! In the same idea as https://github.com/timokau/marvin-mk2/issues/112 ; would it be possible for you to write some documentation for how you test marvin when you develop on it? Creating multiple...
I saw someone in #nixos ask specifically for a reviewer comfortable with C, and it just made me wonder if the ability to summon another reviewer who knows more about...
Hey! This is again an idea that can safely be postponed to after marvin with its current feature set lands. I'm thinking, something similar to marvin would probably make sense...
Based on the comments on https://github.com/timokau/marvin-mk2/pull/101, it sounds like currently marvin's rate-limiting algorithm is based on “involves the person and was updated in the last N days”. This attempts to...
comments by bots such as `nixos-discourse` should not trigger transitions such as #75
In https://github.com/NixOS/nixpkgs/pull/98938#issuecomment-723469357 , the reviewer is inactive, and marvin (correctly) adds a comment that the PR will be set back from `awaiting_reviewer` to `needs_reviewer`, but it doesn't actually do so.