Permanently disable the "Merge" PR button

[rnystrom]

Talking w/ @orta this weekend we had an idea:

Since we want to disable anyone from ever clicking the merge button, how about we setup a simple service and use the GH status API to return state: failure for every single PR? No logic, no nothing, just fail every PR. That way we can stick to using our internal tools and add collaborators w/out having to warn ppl (sorry @Sherlouk @zhubofei 😅 ).

W/ the setup we have, I don't think there is ever a need to click that button.

[Sherlouk]

Don't have write access anyway, so can't press it even if I wanted to!

Sounds like an easy solution to the problem though

[rnystrom]

@Sherlouk 🤔 you should have write-access

[Sherlouk]

@rnystrom I just see this: http://prntscr.com/dth4xm (Buttons are disabled)

[rnystrom]

Hm interesting. I wonder if our tooling messed something up. Going to check internal settings and see what's up there.

[Sherlouk]

Assumed it was intentional to stop me pressing it!

[zhubofei]

I can confirm that the Merge Button was clickable to me for a really short period two days ago and then went grey.

[Sherlouk]

@rnystrom We have #394 which is to use Danger for automated PR verification, could we just add a fail clause for every pull request which is "Merges must be done internally"?

Saves having another service bound to the repo?

[rnystrom]

@Sherlouk totally agree