お題「エンジニア立ち居振舞い」 ということで自分も書きます。
以前、Twitterにつぶやいたのですが
自分のタスクよりメンバーのコードレビューを優先するの、自分の作業は止まるけどトータルで見るとスピードが上がる、と信じたい
— Og🌗エンジニア🏝宮崎 (@koogawa) 2016年7月25日
自分のタスクに着手する前に、チームメンバーのコードレビューをするように心掛けています。
コードレビューが滞ると、次のような悪影響があると思っています。
- 他のメンバーが次のタスクに着手できない
- プルリクエストが消化されないうちにもメインのブランチはどんどん進んでいるのでコンフリクトが起こる
- →コンフリクトを直す手間が増える
- 誰が何をしているか把握しにくくなる
当然、自分の作業は止まってしまいますが、それよりも上に書いた悪影響の方がまずいと思っていて、ここを解決することで全体的には開発スピードが上がると思っています。
共感した立ち居振る舞い
- レビュー依頼はすばやく見る - Sexually Knowing
- 全体的に共感です
- エンジニア立ち居振舞い: プルリクエストは全部見る - 平常運転
- 自分もすべてのコードレビューに目を通すようにしていますが、全部見ることはメンバーには強制していません。
- レビュワーや未来の自分を意識してプルリクエストを作る - stefafafan の fa は3つです
- diffを小さめにするの大事！どんなに多くても500行は超えないようにしたい
