操作
ワークフローに関する質問 #1278
未完了「提案」のパッチをレビューするタイミング
ステータス:
新規
優先度:
通常
説明
「提案」チケットにおいて、提案された機能自体はOKの場合、
添付されているパッチの実装をどのタイミングでレビューするか、
という課題。
- とりあえず無条件に「パッチ適用待ち」にしてブランチに入れてしまって、
そのブランチ上で動作だけではなくコードもレビューするのか - パッチの中身をレビューしてOK判断してから「パッチ適用待ち」にするのか
toshi_a 初音 さんが6年以上前に更新
ステータスには、以下のものがあります。
- 分類待ち: このチケットで議論します。
- パッチ適用待ち: 分類待ちで議論が済んでいるので、パッチをcommitします。このときに議論はしません。
- レビュー待ち: 起票者 がブランチのコミット内容を確認します。第三者は評価しません。
以上のことから、分類待ちのタイミングで添付されているパッチの実装を検証することになります。
Izumi Tsutsui さんが6年以上前に更新
7/15ミーティング時のメモトゥート
https://social.mikutter.hachune.net/@tsutsuii/100378621534585472
そうなると今度は「分類待ち」から「パッチ適用待ち」の移行でコードレビューと仕様レビューとそれぞれが発生するので、モデレータでは手に余る案件も発生することが予想される。そうなった場合の対応として「としぁさん預かり」みたいなステータスを用意して、としぁさんが優先して判断すべきチケットを可視化して優先度を上げて見てもらうべき? という話に。
→ 提案チケットの進め方 に記載のとおり、「提案」のステータス「toshi_aの判断待ち」が追加されました。
Izumi Tsutsui さんが6年以上前に更新
→ 提案チケットの進め方 に記載のとおり、「提案」のステータス「toshi_aの判断待ち」が追加されました。
#1277 に絡みますが、「提案」のチケットで、すでに「実装待ち」にしてしまったものについて、
「実装待ち」から「toshi_aの判断待ち」に遷移できないという問題が。
(具体的には #1255 をとしぁさん判断待ちにしたい)
Izumi Tsutsui さんが6年以上前に更新
cob odo さんは書きました:
「toshi_aの判断待ち」は、まだブランチが必須になっているようです。
遅くなりましたが、OSC京都懇親会会場でとしぁさんと話をして Redmine 設定を修正してもらったので
#1241 を「toshi_aの判断待ち」に変更しました。
操作