Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
レビューのやり方を(ちょっと)整理した話
Search
ken7253
December 16, 2024
Programming
1
510
レビューのやり方を(ちょっと)整理した話
@[IVRyエンジニア忘年LT大会2024](
https://connpass.com/event/333537/
)
ken7253
December 16, 2024
Tweet
Share
More Decks by ken7253
See All by ken7253
CSS polyfill とその未来
ken7253
0
160
Browser and UI #2 HTML/ARIA
ken7253
2
220
PEPCは何を変えようとしていたのか
ken7253
3
430
Browser and UI #1 CSS
ken7253
0
97
オーバーロード関数の話 @Mita.ts #2
ken7253
0
100
フロントエンドカンファレンス北海道参加レポート
ken7253
0
42
カスタムHooksと単体テストの共通点について
ken7253
0
380
検索エンジン最適化はWebサイトのすべてなのか
ken7253
0
55
使いやすいツールチップを実装する方法
ken7253
0
110
Other Decks in Programming
See All in Programming
AI コーディングエージェントの時代へ:JetBrains が描く開発の未来
masaruhr
1
210
NEWT Backend Evolution
xpromx
1
140
テストから始めるAgentic Coding 〜Claude Codeと共に行うTDD〜 / Agentic Coding starts with testing
rkaga
16
5.8k
#QiitaBash MCPのセキュリティ
ryosukedtomita
1
1.5k
バイブコーディング超えてバイブデプロイ〜CloudflareMCPで実現する、未来のアプリケーションデリバリー〜
azukiazusa1
2
640
初学者でも今すぐできる、Claude Codeの生産性を10倍上げるTips
s4yuba
16
13k
“いい感じ“な定量評価を求めて - Four Keysとアウトカムの間の探求 -
nealle
2
12k
MySQL9でベクトルカラム登場!PHP×AWSでのAI/類似検索はこう変わる
suguruooki
1
170
チームのテスト力を総合的に鍛えて品質、スピード、レジリエンスを共立させる/Testing approach that improves quality, speed, and resilience
goyoki
5
1.2k
20250708_JAWS_opscdk
takuyay0ne
2
130
知って得する@cloudflare_vite-pluginのあれこれ
chimame
1
100
AWS Summit Japan 2024と2025の比較/はじめてのKiro、今あなたは岐路に立つ
satoshi256kbyte
1
210
Featured
See All Featured
Speed Design
sergeychernyshev
32
1k
Six Lessons from altMBA
skipperchong
28
3.9k
Distributed Sagas: A Protocol for Coordinating Microservices
caitiem20
331
22k
How to Think Like a Performance Engineer
csswizardry
25
1.8k
CoffeeScript is Beautiful & I Never Want to Write Plain JavaScript Again
sstephenson
161
15k
Cheating the UX When There Is Nothing More to Optimize - PixelPioneers
stephaniewalter
282
13k
The Straight Up "How To Draw Better" Workshop
denniskardys
235
140k
Making Projects Easy
brettharned
116
6.3k
Agile that works and the tools we love
rasmusluckow
329
21k
YesSQL, Process and Tooling at Scale
rocio
173
14k
Balancing Empowerment & Direction
lara
1
480
Designing Dashboards & Data Visualisations in Web Apps
destraynor
231
53k
Transcript
レビューのやり方を(ちょっと)整理した話 @IVRyエンジニア忘年LT大会2024
ブラウザの標準化まわりを追うのが趣味 最近はReactを使ったアプリケーションを書いています。 ユーザーインターフェイスやブラウザが好き。 https://github.com/ken7253 https://zenn.dev/ken7253 https://bsky.app/profile/ken7253.bsky.social https://dairoku-studio.com ken7253 Frontend developer
みなさん!レビューしてますか?
レビューは大変
レビュープロセスを見直す機会があったのでその紹介
レビューコメントのラベル レビューコメントには必ずラベルを付けてもらう運用にしていた。 ラベル 意味 Must 必ず直してほしい箇所 IMO 任意だが自分ならこうする的な意見 NITS 直さなくても問題ないが修正したほうが良い箇所
Question 質問 導入当初は分かりやすくていい印象 チームでコミュニケーションをしていると改善点が見つかる
レビューコメントのラベル 導入からしばらく経っていたのでちょっと整理してみることに 問題点はなんとなく候補があったので、ドキュメントを書いてチームに共有してみた。
問題点 IMOとNITSの使い方が人によって異なる Questionが多用され着地点が分からないコメントがある
改善しよう
話し合うと、ラベルは変更してほしい度合いで使い分けていたことが分かった。 なのでざっくり下記のような対応を行うことに。 ラベルの使い分けを変更要望度を軸としたものに レイヤーが分かれるようにラベルを定義する 質問っぽい指摘を無くす 質問としてコメントされた場合はコードの変更はしない IMOとNITSの変更要望度が人によって異なる IMOとNITSの変更要望度が人によって異なる Questionが多用され着地点が分からないコメントがある
変更要望度に合わせてラベルを整理してみる。 IMOとNITSの変更要望度が人によって異なる
ある程度整理できたので、変更要望度を軸にドキュメントに記載。 ラベル 変更要望度 利用シーン Must 100% 明らかなバグ・仕様と実装の乖離・ガイドライン違反 Want 99%-50% パフォーマンスや可読性など非機能要件的な指摘
IMO / NITS 49%-1% (主観的な)より良い書き方の提案 Ask 0% 質問 どういった場面で利用するのかなどもある程度書いておく。 IMOとNITSの変更要望度が人によって異なる
Questionが多用され着地点が分からないコメントがある
Questionが多用され着地点が分からないコメントがある 純粋な質問まで禁止してしまうのは厳しいのでラベルは残すことになった。 その上で指摘は指摘として他のラベルを使ってもらうように。 難しそうな実装部分はモブレビューを実施してそのログを残す 「大丈夫そうですか?」みたいな変更を期待した質問はNGに レビューコメントの書き方を工夫して「質問っぽい指摘」になるのを避ける
質問を避けるために文章を工夫する方法などもドキュメントにまとめてみた。 書いたこととしては下記の3つの順番でコメントを書くといいよ、という内容でした。 前提として自分の認識 前提が正しかった場合にどのような懸念があるか それを解決する方法 これによって曖昧な部分も自分の認識を前提として変更要望度が出せるように。 質問っぽい指摘を避けるための書き方
まとめ レビューのラベルは変更要望度を基準に整理するといい 質問っぽい指摘は避けてなるべく具体的にコメントを書く