-
Notifications
You must be signed in to change notification settings - Fork 82
翻訳レビュー運用ルール
KenichiroTanaka edited this page Feb 12, 2016
·
3 revisions
PRレビューの基本ルールは以下の通りです。
・PRを出した人は「レビュー待ち」のラベルをつける。 ・「レビュー待ち」ラベルのPR、「再レビュー待ち」ラベルのPR、担当に何もコメントが 無いPR(ラベルのつけ忘れ)は誰がレビューしても良い。(で、マージは本人か、 レビュー者、あるいは他のコミッタが行う) ・何かの理由で、レビュー中であることを宣言したい場合は、「マージは待って (レビュー中)」のラベルをセットしてコメントするか(「○○を確認中です、など」)、 PRにコメントするか、Wikiに書くか、jpug-docに投稿する。 ・レビューが完了したら、レビューが終わったことを意味するコメントを書き(ラベルを 変更するだけでは担当者に通知が行かないため)、特に問題が無い場合は、「マージ待ち (レビュー済み)」「他にも誰か見て(レビュー済み)」のラベルをセットし、コメント (「確認しました」など)する。指摘事項がある場合は、「指摘事項あり」のラベルをセットし、 指摘内容をコメントする。 ・「指摘事項あり」のラベルがセットされた場合、PR担当者は指摘事項に対して対応もしくは コメントを行い、「再レビュー待ち」のラベルをセットする。
ポイントは以下の通り。
・PR作成者を含め2名以上の目が通る
・適宜ラベルをセットする
・ラベルの差し替えだけでは通知が行われないため、何らかのコメントをつける必要がある