- 9:00〜17:45で働きたい
- 夜や休日に働くのが苦手
- 緊急/重要なら夜や休日も働く
- 変に気を遣われるよりは声を掛けてほしい、無理なときは無理と言う
- リモートワークのときは仕事部屋で作業している
- 音楽を流すこともある
- オフィスも好きなのでよく行く
- 集中する早さと長さはオフィスのほうが上、リラックスして働けるのは家のほうが上
- 周りの雑音が気にならない
- 会話でもチャットでも、とにかくみんなと自分が楽しくいれるようにしたい
- チャットより会話のほうが好き
- でもリモートワークがメインの人も多いのでどちらでも気にならない
- DMのような非公開のやりとりはできる限りしない
- メンションは必要なら時間や休みを気にせずいつでも付けてほしい
- 通知するかどうかは自分で制御する
- 逆に相手の都合を気にせず忘れないうちにメンションを付けて伝える
- 嫌な場合は指摘してほしい
- 人と話すのとお酒が好きで、飲み会も好き
- 議事録を取りたい
- 会話だけだと決まったことなどの認識が合わせづらい
- 不要な会議を減らしたい
- 作業時間を確保したいため
- 自分が発言しない会議では内職する
- 情報共有でなく相談の場としたい
- 情報共有は会議で全体へまとめて行わないと齟齬が発生しやすい
- 話すことがなくなったら早めに終わらせたり、頻度を減らしたりしたい
- 作業時間を確保したいため
- 関係値が低かったり、楽しかったりしたら雑談もする
- 素早く実装するのが苦手
- その代わり読みやすくて変更しやすく、不具合の少ない実装が得意だと思う
- レビュー依頼が来たら、できる限り自分の作業を中断して最優先でレビューする
- コードの変更はコミット単位でなくPR単位で確認する
- なのでコミットの粒度やメッセージはレビューしない
- ただしレビュー指摘後の修正はコミット単位で確認する
- 基本的にはローカルで動作確認しない
- 動作確認は作業者の責務だと考えているため
- 誤りがあっても次のPRで直せばいい
- 更新した画面のスクリーンショットを貼ってほしい
- ローカルで動作確認したくないため
- 新規画面の追加はAfterのみ、既存画面の変更はBefore/After両方
- 好みの問題は
[修正任意]を付けて指摘する - 自分で答えを持っている場合はサジェスト機能を使って指摘する
- 自分で書いた指摘は自分で解決する
- マージはPRの作成者に任せる
- 基本的には「読みやすさ」と「書きやすさ（変更しやすさ）」のみ
- 極論をいうと、急にメンバー全員が入れ替わっても開発できるかどうか
- 他の観点はコードレビュー以外で担保すべきと考える
- ビルドできるか → CI
- フォーマットされているか → CI（Linter, Formatter）
- 仕様が正しいか → PO, QA
- 不具合がないか → QA
- ただプロジェクトの性質やフェーズ、メンバーなどによっては他の観点もレビューする
- PRの概要欄やスクリーンショットを見て、変更内容を把握する
- Web上で変更箇所を上から1行ずつ見て、少しでも気になればコメントする
後出しするくらいなら早めに言う
たまたま目に入ったら、レビュー観点以外でもコメントする
- コンテキストが一切わからなくても、PRの概要欄のみを見ればレビューできるように心掛けている
- どんどん指摘してほしい
- 自分の成長に繋がるし、直すなら早いほうがいいため
- 答えを持っている場合はサジェスト機能を使って指摘してほしい
- 指摘でないコメントは読んだら自分で解決する
- できる限りコメントをすべて解決してからマージする
- 基本的には自分でマージするが、他の人がマージしても問題ない
- 自分より他人のために働くほうがモチベーションが上がる
- だけどモチベーションが他人依存なのはよくないとも思っている
- 期待されるとプレッシャーがかかる
- 自己肯定感が異常に低い
- 楽しいかどうかで物事を判断しがち