悪いコードを憎んで人を憎まず！ プルリク送付前に心がけたいコードレビューのコミュニケーション術 - エンジニアHub｜Webエンジニアのキャリアを考える！
悪いコードを憎んで人を憎まず！ プルリク送付前に心がけたいコードレビューのコミュニケーション術 コードレビューを円滑に進め、より学びを促進するために重要な「コードレビュー時のコミュニケーション」について、現役エンジニア・池田 惇さんの経験とともに考えてみます。 アプリエンジニアの池田 惇（いけだ・じゅん／@jun_ikd）です。 コードレビューとは、エンジニアにとって毎日発生する作業であり、「コードを書く」という行為と等しく重要なタスクの1つです。同時に、ただ漠然と「粗探し」をするだけがレビューの目的ではありません。特に若手のエンジニアにとっては、先達のエンジニアのコードにじっくりと触れ、学びを得て、さらにチームに自分の持つ知識・技術を還元する、大事な機会でもあるのです。 今回はコードレビューを円滑に進め、より学びを促進するために重要な「コードレビュー時のコミュニケーション」について、私自
プルリクエストのレビューについて - DMM.comラボエンジニアブログ
こんにちは。 DMM.makeで開発をしておりますコーイチです。 春に入った新人も活躍するようになり、最近業務でプルリクエストをレビューする機会が増えてきました。 日頃どのようにレビューを行っているのかの自分なりの整理と、誰かの多少の参考例になればと 今回プルリクレビューに関して書きました。 レビュー消化ポリシー 差分は全部みる 気になる事柄を残さない 分かっていない事柄を残さない レビュー消化フロー 基本的に私はプルリクエストに対しては2回目を通すようにしています。 【1回目】全部の差分に目を通す 気になったところがあった場合 気になった点が個人の好みの問題である場合 →保留する 気になった点が個人の好みの問題でない場合 →改善案をコメントする よく分からなかったところがあった場合 意味分からん旨をとりあえずコメントする 【2回目】全部の差分に目を通す 保留した好みの問題がまだ気になる場
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
処理を実行中です
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
ログイン - はてな
http://post.simplie.jp/posts/37
crocos.jp
