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
Sponsored
·
Ship Features Fearlessly
Turn features on and off without deploys. Used by thousands of Ruby developers.
→
buntafujikawa
December 02, 2022
Technology
0
500
承認を意識したコードレビュー
コードレビュー LT会 - vol.3 2022/10/12
buntafujikawa
December 02, 2022
Tweet
Share
More Decks by buntafujikawa
See All by buntafujikawa
スクラムとコーチングで作る心理的安全性
buntafujikawa
1
710
ちょっと便利になる自動補完と Ansible でのパッケージ管理について
buntafujikawa
0
800
社内技術広報のすすめ
buntafujikawa
0
190
テストコードについて (仮)
buntafujikawa
0
620
テトリスでイメージする 技術的負債とリファクタリング
buntafujikawa
0
1k
設計レビューのすすめ
buntafujikawa
1
590
Write Code Every Day
buntafujikawa
0
310
ドラゴン桜から学んだ勉強法
buntafujikawa
1
4.9k
How to do effective pair programming with junior engineers
buntafujikawa
1
750
Other Decks in Technology
See All in Technology
AI実装による「レビューボトルネック」を解消する仕様駆動開発(SDD)/ ai-sdd-review-bottleneck
rakus_dev
0
120
JAWSDAYS2026_A-6_現場SEが語る 回せるセキュリティ運用~設計で可視化、AIで加速する「楽に回る」運用設計のコツ~
shoki_hata
0
3k
タスク管理も1on1も、もう「管理」じゃない ― KiroとBedrock AgentCoreで変わった"判断の仕事"
yusukeshimizu
5
2.6k
SRE NEXT 2026 CfP レビュアーが語る聞きたくなるプロポーザルとは?
yutakawasaki0911
1
310
[JAWSDAYS2026][D8]その起票、愛が足りてますか?AWSサポートを味方につける、技術的「ラブレター」の書き方
hirosys_
3
180
マルチプレーンGPUネットワークを実現するシャッフルアーキテクチャの整理と考察
markunet
2
250
Shifting from MCP to Skills / ベストプラクティスの変遷を辿る
yamanoku
4
840
わたしがセキュアにAWSを使えるわけないじゃん、ムリムリ!(※ムリじゃなかった!?)
cmusudakeisuke
1
730
(Test) ai-meetup slide creation
oikon48
3
370
IBM Bobを使って、PostgreSQLのToDoアプリをDb2へ変換してみよう/202603_Dojo_Bob
mayumihirano
1
340
Postman v12 で変わる API開発ワークフロー (Postman v12 アップデート) / New API development workflow with Postman v12
yokawasa
0
120
実践 Datadog MCP Server
nulabinc
PRO
2
190
Featured
See All Featured
Reality Check: Gamification 10 Years Later
codingconduct
0
2k
How To Speak Unicorn (iThemes Webinar)
marktimemedia
1
410
Art, The Web, and Tiny UX
lynnandtonic
304
21k
Code Review Best Practice
trishagee
74
20k
[RailsConf 2023 Opening Keynote] The Magic of Rails
eileencodes
31
10k
Balancing Empowerment & Direction
lara
5
940
DBのスキルで生き残る技術 - AI時代におけるテーブル設計の勘所
soudai
PRO
62
51k
Measuring & Analyzing Core Web Vitals
bluesmoon
9
780
Building Flexible Design Systems
yeseniaperezcruz
330
40k
Building an army of robots
kneath
306
46k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
kevinliebholz
37
6.3k
First, design no harm
axbom
PRO
2
1.1k
Transcript
承認を意識したコードレビュー コードレビュー LT会 - vol.3 2022/10/12 Bunta Fujikawa
About me - 名前 - 藤川文汰 - 現職 - 株式会社コーチェット
(2021年10月〜) - 仕事 - Web エンジニア (Laravel, Next.js など) - 趣味 - 旅行、キックボクシング、カラオケ
今日は承認について話します!
振り返ってみて 今までで嬉しかったコードレビューは?
嬉しかったコードレビュー - 褒めてもらった - 良い学びがあった - 健全な議論ができた - LGTM 画像が面白い
など
嬉しかったコードレビュー - 褒めてもらった - 良い学びがあった - 健全な議論ができた - LGTM 画像が面白い
など 自分の意図を理解してくれる・ 自分の対応した成果や過程を 認めてもらえる
されて嬉しかったコードレビューを 自分ができているだろうか🤔
意識しないと... - 気になるところだけコメント - 自分の意見を一方的に伝えるだけ
承認を意識してコードレビューをする
承認とは - 事実を事実として認めること - 無理に褒める必要はない
褒めるのは難しい 事実を事実と認めることはやり易い
知識があればさらにやり易い
- 承認 + 気になることを伝える - お礼 or 自分が感じたことを組み合わせることが多い レビューでどのように使えるか
レビューでどのように使えるか - 承認 + 気になることを伝える - お礼 or 自分が感じたことを組み合わせることが多い -
存在承認 - スタンプなど何かしらのリアクション - 行動承認 - (バグの修正などに) すぐに対応をしてくれてありがとうございます! - 調査から対応いただきありがとうございました!まとめも見やすいです! - 成果承認 - 〇〇も考慮されたコードで読みやすいですね! - 成長承認 - 前よりも対応のスピードが速くなりましたね! - 以前のレビューで話したことも反映されていて読みやすいですね!
承認の仕方を学ぶ - I / You / We メッセージ - 存在承認
/ 行動承認 / 成果承認 / 成長承認
まとめ - 自分がされて嬉しかったコードレビューをできるようになろう - 指摘ばかりせず良いところを承認しよう - 承認の仕組みを理解すれば挑戦し易い
Author Bunta Fujikawa Twitter @buntafujikawa Qiita @buntafujikawa Github @buntafujikawa