search

sakura-editor / management-forum

管理・運用向けフォーラム（Issues をフォーラム代わりに使う）
2 stars 0 forks source link

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

Open m-tmatma opened 6 years ago

m-tmatma commented 6 years ago

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

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

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

pr-check
k-takata commented 6 years ago

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

m-tmatma commented 6 years ago

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

k-takata commented 6 years ago

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

m-tmatma commented 5 years ago

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

status-check

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

m-tmatma commented 5 years ago

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

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

status-check-2
m-tmatma commented 5 years ago

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

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

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

m-tmatma commented 5 years ago

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