---
# System prepended metadata

title: 2021-12-02 メンターセッション
tags: [Team-2]

---

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

## 質問

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

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

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

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

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

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

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


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

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

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

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