Make-IT-TSUKUBA/alternative-tsukuba-kdb

シラバスデータ更新PRを自動化する

Closed this issue · 6 comments

eggplants commented

現在更新PRが溜まっていますが、レビューが必要な方式だとレビュワーの不在により更新性が損なわれる問題があります。

自動マージはActions導入時権限の関係で諦めたような気がするんですが、自動化したほうが良さそうです。

TODO:

  • src/kdb.json tools/csv/kdb-*.csv に対するテストを書く
  • ラベル automated pr がついているPRがテスト通過しているならreviewなしでauto-mergeされるようにする
onokatio commented

PRじゃなくてpushではダメなんでしたっけ？

onokatio commented

あ〜CIを通したいのか

  • 👍1
eggplants commented

「scheduled workflowとして、テスト通ればmasterにpush、落ちればissueを立てる」とかでもよさそうですね。

eggplants commented

なんにせよ、チェックなしでデフォルトブランチに入れてしまうのは怖いです。

inaniwaudon commented

こちらですが、auto-merge を有効にして PR を作成 → Actions によって PR が立てられた場合は自動で approve を付与することで、テストが通った場合にのみシラバスデータ更新の PR を自動マージするようにしました。これで動いていそうなので、issue は一旦 close したいと思います。

  • 👍1