-
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名以上の目が通る
・適宜ラベルをセットする
・ラベルの差し替えだけでは通知が行われないため、何らかのコメントをつける必要がある