-
Notifications
You must be signed in to change notification settings - Fork 350
Description
内容
test.yml のスナップショット更新の仕組みを、コミットメッセージ検査ベースから workflow_dispatch ベースに変更する提案です。
現状
on: push にブランチ制限がなく、フォークリポジトリで全ブランチに対してワークフローが実行されてしまいます。
また、スナップショット更新はコミットメッセージに [update snapshots] を含めることでトリガーされます。
提案
on: push を branches: [main] に制限し、workflow_dispatch に update_snapshots 入力パラメータを追加します。
スナップショット更新が必要な場合は workflow_dispatch で手動実行します。
Pros 良くなる点
- フォークリポジトリで不要なワークフロー実行を防げる
- ソースコードがシンプルになる
Cons 悪くなる点
- コミットメッセージでのトリガーができなくなる(ただし
workflow_dispatchで代替可能)
実現方法
voicevox_blog で先行実装済みです: VOICEVOX/voicevox_blog#339
実行結果: https://github.com/Hiroshiba/voicevox_blog/actions/runs/21108681739
その他
関連:
- ci: push時のテスト実行をmainブランチのみにしつつ、スナップショット更新を workflow_dispatch ベースに変更 #2888 (ドラフト PR: ブランチ制限のみ追加)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels