/marvin opt-in
/status needs_reviewer

@turion please review.

Binary executes.

/status needs_work

@timokau What am I doing wrong here?

The needs_work status doesn't exist anymore since timokau/marvin-mk2#29. The closest equivalent is awaiting_changes. Most of the time you should just let Marvin do its thing though, you don't have to set the state explicitly. It sets to awaiting_changes when you leave a change-requesting review. Its has recently become a bit more conservative with switching to that state (timokau/marvin-mk2#58), since it doesn't hurt much to have a PR in awaiting_reviewer by mistake but it may hurt a lot to have it in awaiting_changes by mistake.

Marvin should of course have linked the usage document on opt-in, it failed to do so (timokau/marvin-mk2#62).

@timokau Thanks for your detailed explanations.

/status awaiting_changes

@turion Err... what needs to be changed here?

@turion Err... what needs to be changed here?

As I said in #92705 (comment) it would be nice to split into two commits where one contains your refactoring and the other does the version bump :)

Result of nixpkgs-review pr 92705 1

Awesome!

/status needs_merger