"Require status checks to pass before merging" で appveyor のチェックを必須にする

[m-tmatma]

"Require status checks to pass before merging" で appveyor のチェックを必須にしようと思います。
(WIP の GitHub アプリ もついでに必須にする予定)

理由は appveyor が終わらなくてもマージ後 branch を削除すると
sakura-editor/sakura#423 のようにappveyor の結果が
失敗になってしまうためです。

https://qiita.com/yo1000/items/8ffe225716ba3b064697
https://qiita.com/bonotake/items/37fb3194c33f3ae3bbf0

[k-takata]

単に失敗になるのを避けたいのが理由ならば、AppVeyor側で "Always build closed Pull Requests" のチェックを外しておけばよかったりしませんか？

[m-tmatma]

Always build closed Pull Requests は前に試したときにチェックつけてもつけなくても
動作に違いがなかったのですが、動きました?

[k-takata]

実際には試してないので分かりません。

[m-tmatma]

以下チェックをつけました。

Protected Branches機能で柔軟なワークフローを構築する
https://thinkit.co.jp/article/8425

[m-tmatma]

以下チェックをつけました。

master が更新されると、以下のように既存のブランチを更新しつづけないといけないので
OFF に戻しました。

[m-tmatma]

以下チェックをつけました。

master が更新されると、以下のように既存のブランチを更新しつづけないといけないので
OFF に戻しました。

この設定の後、 PR で appveyor のビルドが走らなくなったしまったぽい。

[m-tmatma]

↑ 表示が反映されるのに時間がかかるだけだった。