タグ コードレビュー
人気順 5 users 10 users 50 users 500 users 1000 usersコードレビューでよくお願いする、コメントの追加のパターン7選
2024/11/15
183 users
同僚
追加
Go初学者
前提
コメント
同僚が書いた Go初学者へのコードレビューでよくあったコメント20選 では、Go初学者へのコードレビューでよくあったコメント20選を紹介しました。 今回は私が コードレビューでよくお願いするコメント追加のお願い について紹介します。 前提：コメントを書いて欲しいわけ コードレビューでコメントを書いて欲しい理由は... 続きを読む
Go初学者へのコードレビューでよくあったコメント20選
はじめに こんにちは、ソーシャルベッティング事業本部 海外ベッティング事業部の山崎です。 本記事では、Effective GoやGoogle のスタイルガイド、Code Review Commentsといった公式資料、Future Architectの記事などを参考に、Go を初めて触る開発者を対象にした汎用的なレビューコメントの 20 選を紹介します。 大き... 続きを読む
コードレビュー観点表を作った話
2024/08/07
315 users
コーディングガイドライン
GitHub
ガイドライン
事項
はじめに 今回は、コードレビュー観点表を作った話について少し書かせていただきます。 社内ではGitHubを用いてコードレビューを行っていて、バックエンドの開発においては、コーディングガイドラインも策定しています。 しかし開発において、ガイドラインに書かれている事項が全てではないため、コードレビューを行う際... 続きを読む
TypeScriptの型と値とバリデーション
2024/06/11
144 users
TypeScript
バリデーション
バリデーシ
何度
影響
TypeScript は本質的に自分に型が付与されていると思っているだけの JavaScript です。 いくら型を付与しようが、それが実行時に影響を与えることはありません。 コードレビューをしているとここを誤解している人が本当に多いです。何度も解説しているのですが、なかなか浸透しないので、TypeScript におけるバリデーシ... 続きを読む
sed awk make 互換性 つらい うんざりザリガニ - Lambdaカクテル
2024/04/19
109 users
ザリガニ
Lambdaカクテル
sed
awk
互換性
仕事でsedのコードをパパッと書いて使ったら、コードレビューで動かんと言われてしまった。そういえば、同僚はMacで、おれはLinuxだったな。 まぁsedならよくある話なので、しょうがないと思ってawkで同じことをした。awkは比較的もうちょっと互換性がある気がしているからだ(どっこいどっこいな気もする)。 しかしまだ... 続きを読む
Reactを使ってプロダクト開発している開発者だけでなく、マネージャにも読んでほしい「Fluent React」 - ROUTE06 Tech Blog
2024/03/25
250 users
React
マネージャ
フレームワーク
ライブラリ
プロダクト
チームでReactを使って開発していると、コードレビューをする際に、「この書き方はしない方がいいが、それを説明するには800文字くらい必要。図も描きたい。でもそれらを準備する時間はない。」ということが度々ありました。 また、フレームワークやライブラリの技術選定をする際、マネージャに「どうして技術選定が必要... 続きを読む
良いソフトウェアとコードレビュー / Good software and code review
Scala + Caliban で作るGraphQL バックエンド / Making GraphQL Backend with Scala + Caliban 続きを読む
めんどくさい作業を改善できるようになるには - Konifar's ZATSU
2023/12/21
215 users
Konifar's ZATSU
めんどくさい作業
lint
めんどくさい作業にぶち当たった時、一気に改善してしまう人がいる。ガッと自動化したり仕組みそのものを変えたりしてしまうのだ。「めんどくさい」と心の中で思ったなら、その時スデに行動は終わっているのである。 たとえばコードレビューで都度同じ指摘をしだしたらLintとCIを整備したり、期限のリマインドを何度もし... 続きを読む
長くなりがちだったコードレビューを改善した話 - 弁護士ドットコム株式会社 Creators’ blog
2023/09/08
152 users
Creators’ blog
拡充
タスク
施策
エンジニア
弁護士ドットコム クラウドサイン事業本部でエンジニアをしている山田です。 主にフロントエンドを担当しています。 普段の業務でフロントエンド開発のコードレビューをすることが多く、今回は長い時間がかかりがちだったコードレビューを以下の施策で改善した話をします。 タスクへの認識合わせを拡充 タスクを小さく分... 続きを読む
エンジニアが開発しやすい環境作りをする
2023/04/19
192 users
エンジニア
環境作り
直近
ディレクトリ設計
渋谷
はじめに 自分は渋谷のWeb系開発会社にて執行役員兼エンジニアをやっています。(新卒入社3年目) 直近では6~8名程のエンジニアがいるプロジェクトで、ディレクトリ設計やissue作成、コードレビュー、スケジュール管理、PMへのUI/UX及び機能提案などを行なっています。 その中で自分が「エンジニアチームにとって開発しや... 続きを読む
リーダブルコードの要点整理と活用例まとめ
2023/04/02
141 users
リーダブルコード
要点
要点整理
実務
把握
はじめに 最近コードレビューの機会が増えてきたので、「リーダブルコード」を読み直しました。 リーダブルコードを読んでいく中で要点を整理し、実務の現場でコードを書いたりレビューをする際にどのように活用していくべきかを自分なりにまとめてみました。 この記事を読むことで、リーダブルコードの要点の把握と実際... 続きを読む
コードレビューにラベルを付けるだけでチームの心理的安全性を高めた話
2023/03/06
708 users
ラベル
チーム
齟齬
安全性
必須
ラベルを付けることにより、このような利点があります。 すべてのコメントに対して修正が必須でないので、対等な関係性となる するべきアクションが明確に伝わるため、認識の齟齬が起こらない コメントに重み付けがされるので、優先順位が明確になる 後でやるべきタスクを、タスク化し忘れない コメントの敷居が下がるこ... 続きを読む
コードレビューの「純粋に質問ですが」は「勝手に指摘だと受けとってよくわからん修正するんだろ？質問なんだよ！修正すんなよ！」という意思表示な説
2023/02/14
188 users
指摘
修正
意思表示
殺伐
質問
やきにくのすえなみ @a_suenami コードレビューの「純粋に質問ですが」は表現を柔らかさ目的でなく、「お前は質問だって言わないと勝手に指摘だと受けとってよくわからん修正するんだろ？質問なんだよ！！修正すんなよ！」という意思表示で、むしろ元より殺伐となってる可能性すらあります。 2023-02-13 16:43:45 続きを読む
VSCodeにChatGPTの拡張機能を入れてコードレビューやバグを発見してもらう - Qiita
2023/01/15
394 users
Qiita
vscode
OpenAI
chatgpt
パク
ChatGPTとは？ OpenAIが開発するGPT-3という言語モデルをベースとしたチャットアプリです。 こちらの質問に対して、AIが色々な質問に答えてくれて、一般的な内容だけではなく、コードレビューやバグなども発見してくれるめっちゃ凄いやつです。 細かい内容は以下の記事がとても参考となります。 VSCodeと連携して、ブラ... 続きを読む
コードレビューの生産性を上げるためのTips | Offers Tech Blog
2023/01/10
154 users
Tips
Offers Tech Blog
生産性
藤井
文化
はじめに こんにちは。 プロダクト開発人材の副業転職プラットフォーム Offers を運営する株式会社 overflow のエンジニアの藤井です。 エンジニアであれば誰しも日頃からコードレビューをしたり、されたりしていることと思います。 健全な開発組織を育む意味でもレビューの文化を根付かせることはとても大切ですが、小... 続きを読む
マスク、ツイッターのソースコード凍結。テスラのエンジニアにレビュー指示（Forbes JAPAN） - Yahoo!ニュース
2022/11/08
185 users
テスラ
イーロン
ブルームバーグ
マスク
エンジニア
ブルームバーグによると、イーロン・マスクは、ツイッターの買収手続きを完了して同社のCEOに就任した直後、ツイッターのコードレビューのためにテスラのエンジニアを投入した。 テスラのエンジニアをツイッターの仕事に従事させたことは、マスクが迅速な変化を求めていることの表れだと見られる。 ■ツイッターのチーム... 続きを読む
Twitterの買収を完了したイーロン・マスクがCEOに就任、テスラから集められた50人以上のエンジニアがTwitterのコードレビューに加わる
2022/11/01
100 users
Twitter
イーロン
テスラ
Elon Musk
CEO
2022年10月にTwitterの買収を完了したイーロン・マスク氏が、Twitterの新CEOに就任することが明らかになりました。さらにマスク氏は、自身がCEOを務める電気自動車メーカー・テスラなどのソフトウェアエンジニアを選抜し、Twitterのコードレビューに従事させていると報じられています。 Elon Musk has pulled more than ... 続きを読む
コードレビューのときに見ているところ - 詩と創作・思索のひろば
2022/10/09
231 users
思索
コンテキス
ひろば
モデリング
創作
あるときコードレビューするときにどういうところ見てるんですか？ と訊かれてたしかに自分でもあまり言語化したことはなかったな、と気づいたので簡単に書いておく。 変更意図が要求に沿っているか そもそも実現しようとしていることが、ユーザやプロダクトオーナーの要求に沿っているか。モデリングや実装のコンテキス... 続きを読む
コードレビュー開発者ガイド
2022/02/24
398 users
ドキュメント
プロセス
プロダクト
Google
コード
コードレビュー開発者ガイド はじめに コードレビューとは、コードの作成者以外の人がコードを調べるプロセスです。 Google ではコードとプロダクトの品質を維持するためにコードレビューを実施しています。 このドキュメントは Google のコードレビューのプロセスとポリシーに関する正規の解説です。 このページでは私... 続きを読む
設計/コードレビューで"常に"心がけるポイント - little hands' lab
2022/01/28
392 users
LAB
DDD
little hands
設計
松岡
株式会社ログラスの松岡(@little_hand_s)です。 DDDにおけるドメイン層オブジェクト設計の基本方針[DDD Note] little-hands.hatenablog.com ↑の記事でドメインオブジェクトの設計方針を書きましたが、それ以外の全般的な設計/レビュー観点について書きます。 非常に汎用性のある内容なので、数多くのプログラミング原則... 続きを読む
コードレビューとPull Request、そしてその承認機能の副作用について考える - 時計を壊せ
2021/11/07
161 users
pull request
tl;dr
レビュイー
レビュアー
用語 レビュアー 対象となるコードをレビューする人のことを指します。 レビュイー レビューを受ける人、つまりレビューする対象のコードを書いた人のことを指します。 tl;dr アプリケーション開発業務におけるコードレビューはコードの正しさや質そして一貫性を保ち、それらと同時にコードに対するチームとしての共有知... 続きを読む
全員がOAuth 2.0を理解しているチームの作り方 #devio2021 | DevelopersIO
2021/10/07
344 users
人類
スライド
開発フロー
仕様
コード
DevelopersIO 2021 Decade で「全員がOAuth 2.0を理解しているチームの作り方」というテーマで話させていただきました。 スライド 話した内容 なぜ人類は OAuth 2.0 に入門し続けるのか なぜ OAuth 2.0 をチームに根付かせたいのか 開発フローとしてコードレビューがある 仕様がわからないと、レビューができない コード... 続きを読む
【IMO】コードレビューって難しいよね.pdf
Transcript Copyright© M&A IMO PHP 2021 Akito.Tsukahara Copyright© M&A 2 AkitoTsukahara M&A AkitoTsukahara Copyright© M&A 3 Copyright© M&A 4 Copyright© M&A 5 Copyright© M&A 6 Copyright© M&A 7 Copyright© M&A 8 Copyright© M&A 9 Copyright© M&A 10 IMO Copyright© M&A IMO 11 Copyright© M&A 12 IMO Copyri... 続きを読む
コードレビューの目的と考え方 - osa_k’s diary
2020/10/18
1137 users
損失
議論
osa_k’s diary
コントロール
修正
まえがき コードレビューの目的 大目的 小目的 チェックリスト 優先度高（大きな損失を生む問題・後からの修正が困難な問題） 優先度中 優先度低（システムに大きな影響を与えない問題・後からの修正が容易な問題） レビューを負担にしないために レビューサイズのコントロール 誰がレビューをするか 議論をどうまとめる... 続きを読む
自走プログラマー 【抜粋版】
2020/05/08
112 users
自走プログラマー
抜粋版
ベストプラクティス
本書
各トピック
自走プログラマー 【抜粋版】¶ 2022年05月25日 更新 このサイトについて¶ 書籍『 自走プログラマー 』120の各トピックを抜粋して公開しています。 本書で扱っている問題やベストプラクティスを参照や引用しやすくすることが目的です。 ぜひ、blogやコードレビュー等で引用し、活用してもらえたら嬉しいです。 抜粋版は書... 続きを読む