Maestro merges even if the PR reviewers have requested changes

[natemcmaster]

From @natemcmaster on January 24, 2019 19:10

Is there a way to tell the bot to hold-off merging a PR? I figured the "request changes" gesture in GitHub reviews would have been honored, but apparently not: dotnet/efcore#14506

Copied from original issue: dotnet/arcade-services#103

[natemcmaster]

@markwilkie do we triage issues in this repo? Or should I move this to dotnet/arcade?

[natemcmaster]

From @mmitche on February 7, 2019 17:20

@natemcmaster I'd move it to arcade.

[natemcmaster]

@markwilkie @mmitche can we give this issue consideration? Now that aspnet is enabling automerge, we'd still like a way to tell the bot to hold off merging.

[natemcmaster]

cc @JunTaoLuo @dougbu

[mmitche]

Ya, will see whether this is a quick fix.

[mmitche]

@natemcmaster I think once the PR goes in (and a rollout happens) I'll go and bulk add this to all existing subscriptions.

[natemcmaster]

Awesome 👏