Project

General

Profile

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

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

Added by Izumi Tsutsui 11 months ago. Updated 11 months ago.

Status:
新規
Priority:
通常

Description

「提案」チケットにおいて、提案された機能自体はOKの場合、
添付されているパッチの実装をどのタイミングでレビューするか、
という課題。

  • とりあえず無条件に「パッチ適用待ち」にしてブランチに入れてしまって、
    そのブランチ上で動作だけではなくコードもレビューするのか
  • パッチの中身をレビューしてOK判断してから「パッチ適用待ち」にするのか

History

#1

Updated by toshi_a 初音 11 months ago

ステータスには、以下のものがあります。

  • 分類待ち: このチケットで議論します。
  • パッチ適用待ち: 分類待ちで議論が済んでいるので、パッチをcommitします。このときに議論はしません。
  • レビュー待ち: 起票者 がブランチのコミット内容を確認します。第三者は評価しません。

以上のことから、分類待ちのタイミングで添付されているパッチの実装を検証することになります。

#2

Updated by Izumi Tsutsui 11 months ago

7/15ミーティング時のメモトゥート
https://social.mikutter.hachune.net/@tsutsuii/100378621534585472

そうなると今度は「分類待ち」から「パッチ適用待ち」の移行でコードレビューと仕様レビューとそれぞれが発生するので、モデレータでは手に余る案件も発生することが予想される。そうなった場合の対応として「としぁさん預かり」みたいなステータスを用意して、としぁさんが優先して判断すべきチケットを可視化して優先度を上げて見てもらうべき? という話に。

提案チケットの進め方 に記載のとおり、「提案」のステータス「toshi_aの判断待ち」が追加されました。

#3

Updated by Izumi Tsutsui 11 months ago

提案チケットの進め方 に記載のとおり、「提案」のステータス「toshi_aの判断待ち」が追加されました。

#1277 に絡みますが、「提案」のチケットで、すでに「実装待ち」にしてしまったものについて、
「実装待ち」から「toshi_aの判断待ち」に遷移できないという問題が。
(具体的には #1255 をとしぁさん判断待ちにしたい)

#4

Updated by cob odo 11 months ago

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

#5

Updated by toshi_a 初音 11 months ago

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

#6

Updated by cob odo 11 months ago

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

#7

Updated by Izumi Tsutsui 11 months ago

cob odo さんは書きました:

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

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

#8

Updated by cob odo 11 months ago

Izumi Tsutsui さんは書きました:

cob odo さんは書きました:

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

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

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

Also available in: Atom PDF