プロジェクト

全般

プロフィール

ワークフローに関する質問 #1278

未完了

「提案」のパッチをレビューするタイミング

Izumi Tsutsui さんが6年以上前に追加. 6年以上前に更新.

ステータス:
新規
優先度:
通常

説明

「提案」チケットにおいて、提案された機能自体は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 をとしぁさん判断待ちにしたい)

cob odo さんが6年以上前に更新

一旦 #1241 を「toshi_aの判断待ち」にしようと思ったんですが、ブランチが必須でした。これは意図したものでしょうか?

toshi_a 初音 さんが6年以上前に更新

ブランチのフィールドが有効になってしまっていたので、入力不要にしました

cob odo さんが6年以上前に更新

「toshi_aの判断待ち」は、まだブランチが必須になっているようです。

Izumi Tsutsui さんが6年以上前に更新

cob odo さんは書きました:

「toshi_aの判断待ち」は、まだブランチが必須になっているようです。

遅くなりましたが、OSC京都懇親会会場でとしぁさんと話をして Redmine 設定を修正してもらったので
#1241 を「toshi_aの判断待ち」に変更しました。

cob odo さんが6年以上前に更新

Izumi Tsutsui さんは書きました:

cob odo さんは書きました:

「toshi_aの判断待ち」は、まだブランチが必須になっているようです。

遅くなりましたが、OSC京都懇親会会場でとしぁさんと話をして Redmine 設定を修正してもらったので
#1241 を「toshi_aの判断待ち」に変更しました。

了解です。ありがとうございます。

他の形式にエクスポート: Atom PDF