役立つコードレビュー　8つのヒント | POSTD

2019/02/14 360 users アルゴリズム フィードバック ソフトウェア POSTD コツ

役立つコードレビュー（CR）のコツは、学校では習いません。アルゴリズム、データ構造、プログラム言語の基礎は習っても、確実に役に立つフィードバックを返す方法をじっくりと教えてくれる人はいないでしょう。 コードレビューは優れたソフトウェアを作り出すには欠かせないプロセスです。レビューを通したコードは、そ... 続きを読む

コードレビューは「コードの欠点を指摘する行為」ではない - id:anatooのブログ

2019/01/16 297 users anatoo レビュワー レビュイー 欠点 実力

コードレビューを「コードの欠点を指摘する行為」だと無意識に思っている人を見かけるけども、そういうふうに認識しないほうがチームにとって良いですよ、という話。理由は以下。 レビュワーの方がレビュイーよりも実力が無いといけない、という認識と結びつきがち チームの若いメンバーがレビュワーになりづらくなる 古... 続きを読む

コードレビューを支える『褒め文化』 - エムスリーテックブログ

2018/12/12 269 users エムスリーテックブログ コードレビュー クラウド電子カルテ

コードレビュー、好きですか？ エンジニアリンググループの山口です。 クラウド電子カルテ「エムスリーデジカル」を開発しています。 今回は、チームに根ざしている『褒め文化』についてお話しします。 ※この記事は、エムスリー Advent Calendar 2018 13日目の記事です。 『褒め文化』とは 簡単に言えば、コードレビュー... 続きを読む

コードレビューが好きになるプログラミングの原則 - Speaker Deck

2018/11/26 377 users Speaker Deck プログラミング 原則

Web Developers Meetup Gotanda ～ MC Open Lab. #6 ～で利用した資料です。 https://memberscareer.connpass.com/event/106254/ 続きを読む

コードレビューが好きになるプログラミングの原則 - Speaker Deck

2018/11/25 377 users Speaker Deck プログラミング 原則

Web Developers Meetup Gotanda ～ MC Open Lab. #6 ～で利用した資料です。 https://memberscareer.connpass.com/event/106254/ 続きを読む

Sider 特別インタビュー GitHub アーロン・パターソン氏　 コードレビューにとって大切ことは - Sider Blog

2018/07/26 142 users Sider Aaron Patterson github社

2018年6月12日、13日の2日間にわたり、GitHub Satellite Tokyoが開催されました。 Siderでは、これにあわせて来日していたGitHub社のAaron Patterson（アーロン・パターソン）氏に、特別インタビューをお願いしました！気さくで日本語が堪能なアーロンさんは、終始笑顔で弊社CTOのSoutaroの色んな質問に日本語で答えてく... 続きを読む

無理をしないコードレビュー - クックパッド開発者ブログ

2018/06/19 200 users 無理 コードレビュー クックパッド開発者ブログ 三吉 肩書き

会員事業部の三吉です。 クックパッドでは、GitHub Enterprise の Pull Request を使ったコードレビューを広く実施しています。 この記事では、私がコードレビューすることに対する苦手意識をなくすために意識したことを紹介します。 クックパッドでは、テックリードや新卒、インターン、バイトといった肩書きに関係なく... 続きを読む

コードレビューのベストプラクティス | POSTD

2018/03/06 1358 users POSTD ベストプラクティス コードレビュー かなり やり方

Wiredriveでは、私たちはかなりの数のコードレビューを行います。しかし、ここで働き始める前には私はコードレビューなどしたことがありませんでした。今回は、私がコードレビューをする時に何に注目するようにしているかや、私の考え出したベストなコードレビューのやり方をお話したいと思います。 コードレビューとは... 続きを読む

そろそろコードレビューそのものの必要性について考えるときがきているのかもしれない - タオルケット体操

2018/01/27 412 users SPONSORED LINK プログラミング 文章 仕事

2018 - 01 - 27 そろそろコードレビューそのものの必要性について考えるときがきているのかもしれない 仕事 プログラミング 考えたこと Sponsored link 技術ブログの方に書くか迷ったのですが、かなりポエムの類な文章になりそうなのでこちらに書きます。 ちょっと前にバズったこちらの記事 medium.com に触発されました。 ちなみにコードレビューに関する話としてはまだ僕が色々... 続きを読む

悪いコードを憎んで人を憎まず！ プルリク送付前に心がけたいコードレビューのコミュニケーション術 - エンジニアHub｜若手Webエンジニアのキャリアを考える！

2018/01/24 826 users エンジニアHub GitHub jun 若手Webエンジニア

2018 - 01 - 24 悪いコードを憎んで人を憎まず！ プルリク送付前に心がけたいコードレビューのコミュニケーション術 コードレビューを円滑に進め、より学びを促進するために重要な「コードレビュー時のコミュニケーション」について、現役エンジニア・池田 惇さんの経験とともに考えてみます。 GitHub 池田惇 コードレビュー Tweet アプリエンジニアの池田 惇 （いけだ・じゅん／ @jun_... 続きを読む

コードレビュー なんてしてられるかッ!! // Speaker Deck

2018/01/19 323 users Speaker Deck コードレビュー

All slide content and descriptions are owned by their creators. 続きを読む

コードレビューにおけるレビュアー側のアンチパターン – ITアーキテクトブログ – Medium

2018/01/03 645 users medium GitHub tl;dr レビュイー レビュアー

tl;dr コードレビューが上手く回って無くてチームが疲弊して辛かったよ レビュアーの言い方を変えるだけで大体解決するよ 立場とかで例外を許さず、みんながレビューしてレビューされると良いよ はじめに あるプロジェクトでGitHubのPRベースでのコードレビューを導入をしました。いかんせんチーム開発が初めてレベルの新人さんが多く、何かと苦労しました。特にレビュイーに対して不効率な指摘はそのまま指示の... 続きを読む

コードレビューはコードの責任をレビュアに転嫁する作業 - まなめはうす

2017/11/20 129 users レビュア プログラマ 人格否定 hatenablog.jp

2017 - 11 - 21 コードレビューはコードの責任をレビュアに転嫁する作業 言いたいことはタイトルの通りです。 c5meru.hatenablog.jp コードレビューをつらいと思う人がいるなんて自分には考えもしてなかったので、この記事には驚かされました。指摘を人格否定と思いがちな人がいるのか、人格否定かのような指摘をする人がいるのか、その両方かは分かりませんが、つらいですね。 プログラマ... 続きを読む

コードレビューを受けるのがつらくなったときは - めるノート

2017/11/11 570 users ノート コードレビュー Ano はてな匿名ダイアリー 気持ち

2017 - 11 - 10 コードレビューを受けるのがつらくなったときは そういうときがよくあります。 コードレビューがある会社は今が初めてだけど、きっとこれから先もコードレビューがある限りは、なくならない気持ちなんだと思います。 だから、そんなときに振り返れるようなものを残しておきます。 「コードレビュー つらい」でググってみると、 はてな匿名ダイアリー のこんな記事が見つかりました。 ano... 続きを読む

たのしくなるコードレビュー - クックパッド開発者ブログ

2017/09/22 305 users コードレビュー クックパッド開発者ブログ 負荷 一方 加藤

2017 - 09 - 22 たのしくなるコードレビュー こんにちは！サービス開発部でAndroidアプリの開発をしている加藤（ @k0matatsu ）です。 みなさんコードレビューしていますか？ 最近ではとりいれているチームも多いと思いますが、良い効果をもたらしてくれる一方で、負荷の高い作業でもあります。 また、コードレビュー自体に馴染みの薄かった人はなにをどうしたらいいのか難しいですよね。 ... 続きを読む

入社からの半年間でコードレビューで指摘されたことのまとめ - 30歳からのプログラミング

2017/08/26 1498 users プログラミング 入社 半年間 コードレビュー 備忘録

2017 - 08 - 26 入社からの半年間でコードレビューで指摘されたことのまとめ その他 実務未経験で プログラマ として入社して半年以上が経った。 コードレビューで指摘されたことを備忘録としてまとめておく。 自分なりにまとめたものなので、レビュアーが言いたかったこととニュアンスや解釈がずれている可能性はある。 初歩的な内容ばかりで我ながらうんざりする。 せっかく優秀な同僚ばかりなのだからも... 続きを読む

コードレビューの極意。それは「自分のことは棚に上げる」こと！！ - Qiita

2017/07/14 320 users Qiita 昨日Twitter 極意 遠慮 テスト

はじめに：コードを良くするためなら遠慮は不要 昨日Twitterに投稿した内容が思った以上に拡散されていたので、タイムラインに流れてしまわないようにQiitaにも書いておきます。 コードレビューの極意。 それは「自分のことは棚に上げる」こと！！ たとえば「自分もテスト書けてないしな～」と思って、「ちゃんとテストコード書きましょう」って指摘するのをためらったりしたらダメ！！ そのためにチーム内で「自... 続きを読む

コードレビューを会話しながら行う取り組み - Hatena Developer Blog

2017/06/23 239 users コードレビュー アプリケーションエンジニア itchyny

2017 - 06 - 23 コードレビューを会話しながら行う取り組み こんにちは。アプリケーションエンジニアの id:itchyny です。 今回は、コードレビューを会話しながら行う取り組みについて紹介します。 コードレビューは大事なコミュニケーションの場です。 コードレビューの効用としては、単純なミスがあるコードをリリースしない・プロダクトのコードの品質をよりよくしていく、あるいはその方策を模... 続きを読む

コードレビューのクオリティとスピード，とくにスピードについて，それとコミュニケーションについて - hitode909の日記

2017/06/01 361 users hitode909 ソフトウェア クオリティ スピード 日記

2017 - 06 - 02 コードレビューのクオリティとスピード，とくにスピードについて，それとコミュニケーションについて ソフトウェアを作るときにクオリティとスピードのバランスをとりたくて，どちらかに偏ってはいけなく，どちらもキープしないといけない．すごく雑に *1 とらえると， クオリティ→正しく動き，不具合がないほうがよい スピード→(計算時間ではなく)早く作れるほうがよい ということにな... 続きを読む

先輩から教えてもらったコードレビュー // Speaker Deck

2017/03/08 897 users Speaker Deck 先輩 コードレビュー

LT大会にお呼ばれしました。 内容は以前ブログにしたためた「コードレビューするのが怖いと思っていたエンジニアが半年間コードレビューを経験して思った 10 のこと」についてです☺ http://b.hatena.ne.jp/entry/yutokyokutyo.hatenablog.com/entry/20161213/1481590322 続きを読む

「速」を落とさないコードレビュー

2017/01/28 819 users コードレビュー Rails onk 株式会社ドリコム 趣味

「速」を落とさないコードレビュー 1. 「速」を落とさない コードレビュー 2017-01-28 Forkwell Meetup #3 Productivity Engineering 大仲 能史 a.k.a. @onk 2. 自己紹介 • 大仲 能史 a.k.a. @onk • 株式会社ドリコム 11年目 – アプリケーションスペシャリスト – Rails 歴が一番長くて 8 年ぐらい • 趣味... 続きを読む

コードレビューをより効果的にする方法

2016/11/22 153 users コードレビュー フォーマット 方法 パク 着眼点

重要なポイント コードレビューを行うとき、本当に多くの着眼点がある。チームは何がこのプロジェクトで大切なのかをきちんと特定し、レビューの間に一貫してこれらの確認を行わなければならない。 人間によるレビューはとても貴重なので、できることは全て自動化する必要がある。例えば、フォーマット、スタイル、よくあるバグに対する確認、よくあるのセキュリティ課題の特定、自動化されたテストの実行など。 パフォーマンス... 続きを読む

エンジニア立ち居振舞い：重箱の隅をつつかない - 面白コンテンツ探求日記

2016/11/12 180 users 重箱 エンジニア立ち居振舞い 指摘 面白コンテンツ探求日記

2016 - 11 - 12 エンジニア立ち居振舞い：重箱の隅をつつかない お題「エンジニア立ち居振舞い」 自分は重箱の隅をつつかないというのを意識してる。 重箱の隅をつつく問題はコードレビューの現場でよく聞く。レビューの場で所詮「書き方レベル」の指摘が横行してしまうというやつ。 誰しも綺麗なコードを追求したい気持ちはあると思うし、自分もそうなんだけど、あまり良くないなと思ってやらない事にしてる。... 続きを読む

【GitHub】1年以内に作成された人気リポジトリ20個をまとめました | HRナビ by リクルート

2016/10/24 478 users GitHub Bootstrap リポジトリ リクルート 長年

コードレビューや情報収集など、エンジニアの開発シーンに欠かせないGitHub。 技術者の興味関心や開発トレンドが詰まったサービスなので注目が集まることも多々。今年に入ってからは FreeCoodCamp の総スター数が、長年トップだった bootstrap を上回ったことが話題になっています。 今回は、2015年9月30日〜2016年9月30日に作成されたリポジトリのなかでスター数が多いリポジトリ... 続きを読む

［速報］GitHub、コードレビューのための新機能を発表。コードの行ごとや課題ごとにコメント可能。GitHub Universe 2016 － Publickey

2016/09/14 184 users GitHub Publickey サンフランシスコ 基調講演

GitHubはイベント「GitHub Universe 2016」をサンフランシスコで開催しています。 初日の基調講演に登壇した同社CEO兼共同創業者のChris Wanstrath氏は、新機能のCode Reviewを発表しました。 Chris Wanstrath氏。 Pull requestはGitHubを使う上で核となる機能だ。Pull Requestはコードに関するコラボレーションの方法と... 続きを読む