コードレビューの生産性を上げるためのTips | Offers Tech Blog
はじめに こんにちは。 プロダクト開発人材の副業転職プラットフォーム Offers を運営する株式会社 overflow のエンジニアの藤井です。 エンジニアであれば誰しも日頃からコードレビューをしたり、されたりしていることと思います。 健全な開発組織を育む意味でもレビューの文化を根付かせることはとても大切ですが、小規模な組織の場合「レビューアが足りない」という問題が往々にして起こり得ます。 もちろん、特定のエンジニアにコードレビューが集中してしまうのを防ぐために、チーム全体で負荷分散を図るのが本質的かつ王道的なアプローチではあります。 しかしときには、とにかく個人の力で乗り越える、という状況も避けられないでしょう。 そこで今回はコードレビューの生産性を上げるための Tips をいくつかご紹介します。 自分でも開発をしなければならないが、その片手間で一日に何本ものプルリクエストを確認しなけ
コードレビューガイドラインを策定して継続的インテグレーションを実現する
ヤフー株式会社は、2023年10月1日にLINEヤフー株式会社になりました。LINEヤフー株式会社の新しいブログはこちらです。LINEヤフー Tech Blog こんにちは。Yahoo!広告 ディスプレイ広告エンジニアの下村です。広告配信システムの開発を担当しています。 ヤフーの広告システムは巨大なトラフィックを捌いていることやステークホルダーが多いことから、小さなバグも大きな事故になり得ます。加えて、目まぐるしく変わる法律やステークホルダーの方々からいただいた要望への対応を迅速に行うため、日々の開発速度も高い水準が求められます。しかし、以前は一度のPull Request（以下PR）に含まれる変更量が多く、レビューの時間が長引いて機能開発に時間がかかったり、大規模なコンフリクトが発生してバグが混入しやすい状態になっていました。そこで、継続的インテーグレションのプラクティスの適用を目指した
コードレビューのときに見ているところ - 詩と創作・思索のひろば
あるときコードレビューするときにどういうところ見てるんですか？ と訊かれてたしかに自分でもあまり言語化したことはなかったな、と気づいたので簡単に書いておく。 変更意図が要求に沿っているか そもそも実現しようとしていることが、ユーザやプロダクトオーナーの要求に沿っているか。モデリングや実装のコンテキストを自分でも把握しておく。 関連する別の変更やイシューなど、自分が知っていて相手が知らない有意義な情報があったらコメントする。 モデリングが妥当か モデルによって意図が表現できているか。仕事が適切な粒度で明確に切り分けられているか。意図のない共通化がなされていないか。 わかりやすい名前がつけられているか。ここが混乱していると何かがよくないサイン。既存のコードがすでに……ということもある。そういう場合は改善できそうな道筋について議論できるとベター。 仕事にあったインタフェースになっているか。テスト
17年間のCTO経験から得た「経営者としての視野の拡げ方」｜MIDAS Technology Review
今回はグリー株式会社（以下、グリー）取締役 上級執行役員 最高技術責任者（CTO）兼 デジタル庁 CTOを務められる、藤本真樹さんにお話を伺いました。藤本さんは、2005年にグリーに取締役として参画されてから約17年間、CTOとして技術面から経営に携わっています。 そこで、藤本さんの17年間の経営者経験を振り返り、「経営者としての視野の拡げ方」というテーマでお話を伺いました。本記事では、そこで藤本さんに語っていただいた内容をお伝えします。 グリー 取締役 CTO 藤本 真樹 氏グリー株式会社 取締役 上級執行役員 最高技術責任者（CTO）兼 デジタル庁 CTO 藤本真樹さん（＠masaki_fujimoto） 上智大学文学部卒。2001年、株式会社アストラザスタジオ入社。2003年、有限会社テューンビズ入社。PHP等のオープンソースプロジェクトに参画し、オープンソースソフトウエアシステムの
自分がコードレビューで気をつけていること（2020.5 iOSアプリ開発版 ）
わたくし@yimajoが仕事のコードレビューで気をつけていることを書いておきます。特にiOSアプリ開発をしているので内容はiOSアプリ開発におけるコーディングやStoryboardについて書いていますが、その前提やメンタル面の話が多めです。 ちなみにiOSアプリのコードレビューで見ているポイント 2020年5月版にインスパイアされて書いてみました。 前提 レビュー時にリスクや課題、問題という言葉を意識して使い分ける 「このコードは問題です」という指摘をするとき、「問題」という言葉の意味をなるべく共通の理解を持つ 問題 現在起こっている正常でない状況 例: 仕様を表現できていない 例: 不具合がある 課題 問題を整理/分割したもの 例: deprecatedなAPIを使っている リスク 時間の経過もしくは何かの要因によって課題や問題になってしまう事柄 例: テストコードが書けていないので手を
フロントエンドエンジニア１年目はコードレビューでどんな指摘を受けるのか - Qiita
社内のコードレビューで先輩から指摘いただいた事をまとめました。 主にJavascript（Vue.js）のコードです。 本記事の修正前のコードは、（Qiita用に簡略化してますが）実際に自分が書いたダメコード。 ちゃんと失敗を活かしていけよと自分へのメッセージを込めて書きます！ 筆者のスキル感 エンジニア歴（≒プログラミング歴）ちょうど１年 フロントエンドエンジニア Vue.js/Nuxt.jsで開発 凡ミス 一番指摘されると申し訳なく感じるところ。 凡ミスがマージされるとバグの温床になるので、気をつけなきゃですね・・。 デバッグの痕跡が残っている console.log('hoge') 現プロジェクトはgitのpre-commit hookを使ってをコミットしないようしてるのですが、 そこをすり抜けてデバッグコードを潜ませてしまう事があります・・。 タイポ const enviromen
GitHubのReview機能が強化された - Qiita
株式会社オズビジョンの@terra_yuccoです。 本日、いつものように貯まっているPull Requestを見ようとGitHubの画面を見たところ、 Discussionの見え方が違い「？」と思っていました。 そこでチームメンバーから「GitHubのレビュー機能がめっちゃパワーアップしました。」と 共有されたGitHubのリリースノート？を読んだので、Tipsを共有します。 Tips Pull Requestに対して、総評という形でReviewを書けるようになった Reviewには以下の種別を付けられる Comment: 軽微な指摘、コメント Approve: 承認 Request Change: 修正依頼 Request Changeが付いたPull Requestは、AdminもしくはRepository Owner以外は、 修正しないとMergeできない Pull Request
「速」を落とさないコードレビュー
Forkwell Meetup #3 https://forkwell.connpass.com/event/48147/
コードレビューをする話を新卒研修で話しました。 - パルカワ2
今いる会社には、新卒研修で座学と呼ばれるものがあって、先輩エンジニアがある程度好き勝手に話したいことを1時間ほど話す時間があります。 そんな最高の学習環境である座学で、コードレビューについて話してきました。 テーマは、 https://github.com/kenchan/keynote-theme を使いました。 内容についての補足 iOSとか出来ないので、そのあたりの説明が出来ない 最近コードレビュー全くしてない 相手によってはLGTM画像が嫌いな場合がある TPOがあるよね コンテキストが違う人にいきなりはキビシイ 海外の人にアニメ画像は使わないほうがよさそう 英語だと横向きの顔文字とかよく見る ;P 美女LGTM画像は会社で使ったことない スライド作る時に気をつけたこと 6個のステップ、3つの目的とか個数を言うようにした。 新卒氏たちに今やるべき事を認識してもらう 新卒氏たちにも出
まだ「マンガ大賞」などの賞に選ばれてない漫画でオススメ2016 - 根室記念館
■数々の名誉ある賞 「次にくるマンガ大賞」「マンガ大賞2016」「このマンガがすごい！」など、今年も数々のマンガランキングが開催され、多くの魅力的な漫画が受賞しました。中でも全ての賞に選ばれ、「マンガ大賞2016」では見事に一位に輝いた「ゴールデンカムイ」は文句無しにオススメできます。 何と言っても作者が賞に選ばれる度に、自画像を作中でもトップクラスのサイコ野郎にしてコメントする所が最高です。多くの紹介記事では「アイヌ特有のグルメがすごい！」「不死身の主人公の活躍がカッコいい！」と評されていますが、この漫画の真の魅力は、他の追随を許さない狂った死刑囚が次々登場する上に、そのサイコ具合が巻を増す毎に加速していく部分にあると思っています。 ヤンジャンの中でもかなり愉しみにしている作品なので、「マンガ大賞2016」の選評の一部が非常にふわふわしており、作中の魅力が何一つ伝わらない事に苦言を呈した
Androidソースコードレビューで指摘する事が多い項目まとめ - こやまカニ大好き
業務でソースコードレビューを行う機会が増えたので、複数回指摘した項目や気になった実装などをまとめてみました。 こういう観点をできる人と共有できるといいなあ…。 2014/09/29 23:00 一部修正しました。 業務上ソースコードレビューの名目で仕様・デザインまで見ることになっていたためこれらを先頭に書いていましたが、わかりづらかったため最後にまとめました。 Fragment関連 FragmentとActivityの密結合 Fragmentが特定のActivityから呼ばれることを想定して書かれている場合、そのFragmentとActivityは密結合である場合が多いです。 具体的には、以下の様な実装です。 ActivityのViewを参照する Activityのメソッドを直接呼び出す なぜダメか Fragmentの利点のひとつは優れた再利用性にあります。 Fragmentが特定のAct
デキるプログラマだけが知っているコードレビュー7つの秘訣
SonicGarden Study #11で放送された資料から一部スライドを抜いたものになります。 http://sonicgarden.doorkeeper.jp/events/13229 ----- 優れたプログラマだけが優れたソースコードを書くことができます。 では優れたプログラマになるにはどうすれば良いでしょうか。 自分の書いたコードを、優れたプログラマに指摘してもらうことが一番の近道です。それがコードレビューです。たった一人でコードレビューも受けずに、ただ書き続けてもクソコードはクソコードのままなのです。 そこで今回は、良いコードが書けるプログラマになるための、コードレビューを上手に実践する秘訣を話します。Read less
中規模iOS開発とコードレビュー
フリマアプリFrilのリニューアルを題材に、iOS開発でのコードレビュー事例を紹介します
バナナ学園純情乙女組『秋の大運動会』 - 青春ゾンビ
男女問わずセーラー服をまとった50人の若者が、爆音ノイズミックスで、アニメソング、初音ミク、モー娘。、ももいろクローバー、AKB、サンボマスター、小沢健二・・・ジャンク混ぜこぜの音楽で踊り狂う。「一億年と二千年前から君が大好きだと叫びたい、実行せよ！恋愛レボリューション21、世界はそれを愛と呼ぶんだぜ！だから愛し愛され生きるのさ！」って感じ。壇上のセーラー戦士達は様々な格好に着替えてパフォーマンスを行う。スクール水着、ブルマ、ロリータ、SM、ヲタ芸、『ワンピース』『テニスの王子様』『けいおん！』『新撰組』『パイレーツ・オブ・カリビアン』『ケロロ軍曹』『新世紀エヴァンゲリオン』『少女革命ウテナ』などのコスプレ。この10年で日本を席巻したカルチャーを飲み込んで、全部まとめて塊として吐き出す。 携帯電話を放り投げ、糸電話を使い大声でコミュニケーションを切望する彼や彼女にグッときてしまう。情報過多
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
アプリレビュー依頼先一覧（iOS＆Android） - Sawalog