# 2021-12-02 メンターセッション

## 質問

### 普段どのようなルールでコードレビューをするか

#### レビュー頻度
- 1時間ごとにGithubを確認してレビュー依頼がないか見るというルールを作ったこともある
- あまり評判が良くなかった

#### 誰がレビューするか
- 持ち回りで買えることが多い（毎回2~3人くらいは欲しい）
- 強い人だけが見るとその人に甘えてしまう
- 強い人だけが疲弊していく

#### どんなふうにレビューするか
- 観点を分ける
- 仕様に関するところ
- 書き方等の設計に関するところ

#### 工夫できるところ
- 同じような話になると話が進まない
- 致命的じゃないなら一旦マージしよう
- 口頭で集まって確認して認識合わせる方が後々楽なのではないか
- 指摘の内容を周知することが大事

### デザインを後からもらう意図はなんだったのか

- デザインが見えると作業になりそうという懸念があった
- 結構シンプルなのでデザインがモデリングに影響を与えそう
- 進め方として最初にモデリングやデータベース設計について話し合う時間を取ってほしかった

### チーム開発のコードレビュー

#### バックエンド
- ClientのIDにfirebaseのuidをそのまま使うのは良くなかった

#### フロントエンド
- ちらつきを防止するためにはSSRを使うと良い

###### Tags: `Team-2`