Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

オートマージ及びマージキューを有効化する #2457

Closed
Hiroshiba opened this issue Jan 1, 2025 · 5 comments
Closed

オートマージ及びマージキューを有効化する #2457

Hiroshiba opened this issue Jan 1, 2025 · 5 comments

Comments

@Hiroshiba
Copy link
Member

内容

このリポジトリで試験的にauto mergeとmerge queueを有効化します。
レビューしたあとマージするためにテストが通るまで待機する必要があったのが自動化されたり、最新のmainブランチだとエラーになってしまうのに誤ってマージしてしまうミスを防ぎます。

こちらのプロジェクトの検証も兼ねています。

コミッターの体験は変わらないはずですが、なにか意図しない挙動をしたりするかもしれないので、あらかじめissueを作りました。

Pros 良くなる点

レビューがかなり楽になる

Cons 悪くなる点

PRごとのCIステータスが増えたり、実行されるワークフロー数が増える

実現方法

その他

@Hiroshiba
Copy link
Member Author

がマージされたので設定していきます!

マージキューを有効にできるのはmainブランチだけなので、これだけstatusチェックされるようになります。
それ以外でプロテクトしていたブランチ、release-系とproject-系は以前と同様に2名以上のレビューが必要のままです。
(大体force mergeになると思われます)

@Hiroshiba
Copy link
Member Author

設定しました!

@Hiroshiba
Copy link
Member Author

無事エディタのmainブランチにマージされました!!!

@Hiroshiba
Copy link
Member Author

あとマージキューのたびに2回テストが実行される部分がなんとかなればcloseで良さそう!!

の有効性を確認すれば完了なはず。

@Hiroshiba
Copy link
Member Author

Hiroshiba commented Jan 5, 2025

ちゃんとCIが1つだけ動いてて、ちゃんと見てくれてそう!!
https://github.com/VOICEVOX/voicevox/actions/runs/12617168760/job/35159170960

Start processing validator: merge_gatekeeper....
7 out of 7

Total job count:       7
Completed job count:   7
Incompleted job count: 0
Failed job count:      0
Ignored job count:     0

このissueは完了な気がします。
もうちょっとだけ運用してみて固まったら他のリポジトリにも適用していきたい!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant