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

ci: upsidr/merge-gatekeeperを追加 #6

Merged
merged 3 commits into from
Jan 1, 2025

Conversation

sevenc-nanashi
Copy link
Member

@sevenc-nanashi sevenc-nanashi commented Jan 1, 2025

内容

upsidr/merge-gatekeeperを付け足します。

関連 Issue

スクリーンショット・動画など

(なし)

その他

READMEも更新しようと思いましたが、どちらかと言えばvoicevox-project/docsにあるべきだな~って気持ちがしたのでしてません。
あとこれの失敗チェックもしたい。

@sevenc-nanashi
Copy link
Member Author

@Hiroshiba 一旦これを強行マージできるか試して置いた方がよさそう?

@Hiroshiba
Copy link
Member

Hiroshiba commented Jan 1, 2025

PRありです!!!
なるほど、失敗するかをチェックするのに今のmainブランチを利用する感じですかね?

変更しないといけないのが2箇所ほど!

たぶんjob名をmerge-gatekeeperにしないとダメかもです。
ついでrule側でチェックするstatus名も-にしないとかも。

あとタイムアウトがデフォルトで600秒っぽいので、RulesetのStatus check timeout (minutes)に設定予定の300分に合わせて、300*60の18000を指定しておきたいかも!

READMEの変更は間違えて古いのをコピーしてしまいそうなので、やっといたほうが良さそうだなと感じました。
けどまああとでチェックのために追加PRでも良さそう!

@Hiroshiba
Copy link
Member

あ、force mergeのチェックのためですね!! 確かに。
たぶんRulesetのbypassにメンテナを指定すれば、メンテナは強行マージできる気がするので試してみようと思います 🙏

@sevenc-nanashi
Copy link
Member Author

あ、強行マージというより独断マージですね。

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

だけどapproveせずbypassでマージしてみたいと思います!!

token: ${{ secrets.GITHUB_TOKEN }}
self: merge_gatekeeper
# https://github.com/upsidr/merge-gatekeeper/issues/71#issuecomment-1660607977
ref: ${{ github.event.action == 'auto_merge_enabled' && github.event.pull_request.head.ref || github.ref }}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

こんな感じにしたりすると、「PRのとき」という意図が伝わりやすいかも?

Suggested change
ref: ${{ github.event.action == 'auto_merge_enabled' && github.event.pull_request.head.ref || github.ref }}
ref: ${{ github.event.pull_request && github.event.pull_request.head.ref || github.ref }}

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

確かに。そうしてみます。

@Hiroshiba Hiroshiba merged commit 2805442 into VOICEVOX:main Jan 1, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants