Project

General

Profile

Proposal #103

Code Reviewの実体はチケットにした方が良いのではないか

Added by Haru Iida about 11 years ago. Updated about 11 years ago.

Status:
終了(Closed)
Priority:
通常(Normal)
Assignee:
Target version:
Start date:
06/27/2009
Due date:
% Done:

100%

Estimated time:

Description

コードレビューの1レコードには現在以下の2種類のデータを保存している。

  • レビュー内容、コメント、ステータス、レビュー者などのレビュー情報
  • ソースコード、リビジョンなどのレビュー対象物情報

このうちレビュー情報はコードレビュー内に持つのはやめて、チケットにしてしまえばよいのではないかと思い付いた。レビューのレコードにはチケットへのリンクを持つのみにする。

これによって
  • ワークフローを定義できる
  • カスタムフィールドが使える
  • watcherを指定できる
  • バージョンとの関連付け
  • 工数管理
  • 検索ができる

などなど、とても便利になる。

また、メール送信や活動ページへの表示、ステータス管理などをチケットに任せられるのでCode Review Plugin側のコードを削除できる。

ただ今までのバージョンとデータの互換性がなくなるんだよね。
最初からそうすればよかった。


Related issues

Related to Code Review - Feature #114: 修正のコミットに連動してレビューをクローズしたい終了(Closed)Haru Iida07/02/2009

Actions
Related to Code Review - Feature #33: レビューが追加されたらマイページで表示できるようにする。終了(Closed)Haru Iida05/11/2009

Actions
Related to Code Review - Feature #119: 旧レビューデータを新データにコンバート終了(Closed)Haru Iida07/14/2009

Actions
Related to Code Review - Feature #120: レビューをどのトラッカーに割り当てられるか選択できるようにする終了(Closed)Haru Iida07/14/2009

Actions
Related to Code Review - Feature #123: レビューの題名を付けられるようにする終了(Closed)Haru Iida07/14/2009

Actions
Related to Code Review - Feature #125: レビュー画面で状態を変更したらチケットの状態を変える終了(Closed)Haru Iida07/15/2009

Actions
Related to Code Review - Feature #124: レビューとチケットを相互に行き来できるようにする終了(Closed)Haru Iida07/15/2009

Actions
Has duplicate Code Review - Feature #5: レビューとチケットの関連付け終了(Closed)Haru Iida04/30/2009

Actions
#1

Updated by Haru Iida about 11 years ago

  • % Done changed from 0 to 40

取りあえずレビューのチケット化はできた。
またメールは飛ばない。ステータスの変更もレビュー画面からはできない(チケット画面からはできる)

#2

Updated by Haru Iida about 11 years ago

  • % Done changed from 40 to 50

メール送信もできた。
次は状態変更。

#3

Updated by Haru Iida about 11 years ago

  • Status changed from 新規(New) to 解決(Resolved)
  • % Done changed from 50 to 100

状態変更は別チケットにしてこれは解決にする。

#4

Updated by Haru Iida about 11 years ago

  • Status changed from 解決(Resolved) to 終了(Closed)

Also available in: Atom PDF