VOICEVOX / voicevox_project

VOICEVOX内のプロジェクトを管理するリポジトリ
15 stars 3 forks source link

テストが通ったら自動的にマージできるようにしたい #53

Open Hiroshiba opened 4 months ago

Hiroshiba commented 4 months ago

内容

テストが通ったら自動的にマージできるようにしたいです。

merge queueを使えば可能なはずなのですが、VOICEVOXのレビュー&approve事情に合ってないためそのままでは適用できなさそうでした。

VOICEVOXのプルリクエストマージは、以下の2つの条件のうちどちらかを満たすと可能としています。

branch protectionを使えば「何人以上のapproveが必要か」と「誰ならforce mergeが可能か」を設定できます。 なので、2人以上のapproveがあるか、@maintainerチームのforce mergeを有効にすることで、↑のどちらの条件でもマージできるようにしています。

問題はこれがmerge queueで使えないっぽいことです。 というのも、「N人以上のapprove」ではmerge queueを使えますが、force mergeの場合はmerge queueを無視して対象のブランチにマージするという挙動をするっぽいです。

さてどうしようかなという感じです・・・ 😇

実現方法

いくつか解決策はあると思います。とりあえず列挙してみます。

その他

メモです。

Hiroshiba commented 4 months ago

もし挑戦してみたい方がいらっしゃったらコメントください 🙇 条件等も細かいところが分かりづらいと思うので何でもお聞きください!

Hiroshiba commented 4 months ago

@VOICEVOX/maintainer すみません、ちょっと勝手にプロジェクト作ってしまいました 🙇

オートマージしたく色々試してみたのですが、github の仕様とVOICEVOXの要件が合いませんでした・・・。 このプロジェクトリポジトリに作るissueとしてはちょっと規模感が小さいかもなので、移動した方が良さそうだったら移動します 🙇