レビュー体制
レビューをうけるひとのすること
サイト制作課題
#frontend_review のタグをつけて toot
React課題
#frontend_review #react_review 二つのタグをつけて toot
レビュアーのすること
- コース修了者は修了直後から1次レビュワーとして参加
- 週あたり1次レビュワー2人・2次レビュワー1人
- 週が変わるタイミング(月曜日0:00)で交代
- 48時間以内に見る
- 担当週に稼働できなさそうであれば事前に相談する
- 現状react課題は2次レビューのみ
- サイト制作課題の1次レビュアーは #frontend_review タグを巡回する
- React課題や2次レビューの修正を見つけたらReact課題のレビュアーや2次レビュアーにつなぐ
- 2次レビューは週が変わった場合でも担当者そのまま
- 修正後のレビュー依頼は前回の2次レビュアーに対してメンションしつつ投稿する
現在のシフト
二次レビュー(コース共通)
moai → higashiji
サイト制作課題
tomu → taoka → yae → minami → wakana → ren → tomonari → shunya → koga
react課題
chinatsu → mutsuki → keisuke → haru
端まで行ったら最初に戻る
レビュワーの都合により順番が前後することもあり
レビュー方針
以下の項目を満たしていればOK
- figmaとの差異がない
- gitの使い方を理解している
- 余計な差分が出ていない
- prettierが効いている
- 画像が最終課題リポジトリにもともと入っている素材を使っている
- src/img/
- headerのアイコン等不足しているものもあるかも
- 上記に含まれない内容は、修正依頼ではなくヒントとしてコメントする
- ヒント例
- クラス名が適切か
- htmlタグが適切か
- より簡潔な書き方はないか
- bootstrapを使うことを推奨しなくてもよい
- 以前の課題ページではbootstrapのgridを使うことを推奨する記述があったが、むしろ使わない方が楽な場面が多そう
- 難しい場面ではヒントを与えてもよい(githubのsuggestion機能、スクリーンショット等)
- goodな実装があれば褒める
- 参考になる記事があれば貼る
- 自分でも理解が曖昧な点があれば、いい機会なので調べる
- 課題に直面した際、時間があればdawnでペアプロをするのが最高
- 具体的な実装の背後にある考え方を説明する
PlayGround公式レビューガイドライン
レビューガイドライン
参考
新人プログラマをレビューで殺さない方法 - Qiita
レビューで大量の指摘をして大きな手戻りを発生させた原因はレビューアの私にあった - Qiita
[GitHub] Multi-line code suggestionsでコード提案機能が便利になりました | DevelopersIO