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
buntafujikawa
December 02, 2022
Technology
0
380
承認を意識したコードレビュー
コードレビュー LT会 - vol.3 2022/10/12
buntafujikawa
December 02, 2022
Tweet
Share
More Decks by buntafujikawa
See All by buntafujikawa
スクラムとコーチングで作る心理的安全性
buntafujikawa
0
540
ちょっと便利になる自動補完と Ansible でのパッケージ管理について
buntafujikawa
0
600
社内技術広報のすすめ
buntafujikawa
0
110
テストコードについて (仮)
buntafujikawa
0
490
テトリスでイメージする 技術的負債とリファクタリング
buntafujikawa
0
790
設計レビューのすすめ
buntafujikawa
1
500
Write Code Every Day
buntafujikawa
0
260
ドラゴン桜から学んだ勉強法
buntafujikawa
1
4.2k
How to do effective pair programming with junior engineers
buntafujikawa
1
580
Other Decks in Technology
See All in Technology
Combineを中心とした処理をSwift Concurrencyへ (これまでも調べた調査と向き合い)
fumiyasac0921
1
180
庁舎DX推進ガイドブック
senryakuka
0
1.8k
GitHub Actions Runner Controller
takesection
0
110
DevOps Topologies 10 years on: what have we learned about silos, collaboration, and flow? - Matthew Skelton, Conflux
matthewskelton
PRO
2
420
小さく始めるAnsible
stopendy
0
210
OCI Data Science Service 製品概要
oracle4engineer
PRO
0
110
滑空スポーツ講習会2023 航空安全講習会 第4回 日常整備に役立ちそうな雑情報 / JSA Safety Seminar 2023 glider maintenance
jsaseminar
0
110
プロダクト開発ゼロイチの分類とロジックス事業がイチに至るまで
niwatakeru
0
100
Exadata Database Service on Dedicated Infrastructure(ExaDB-D) UI スクリーン・キャプチャ集
oracle4engineer
PRO
0
1.1k
SwiftUIのpropertyWrapperをふんわり理解する
jambo_develop_team
0
110
進化する事業とデータ構造 ~Cloudbaseの場合~
tockn
2
230
SmartHR プロダクトエンジニア求人ガイド 2024上期
smarthr
0
140
Featured
See All Featured
10 Git Anti Patterns You Should be Aware of
lemiorhan
644
57k
Building Flexible Design Systems
yeseniaperezcruz
317
37k
Docker and Python
trallard
33
2.6k
What's in a price? How to price your products and services
michaelherold
236
11k
Stop Working from a Prison Cell
hatefulcrawdad
265
19k
Dealing with People You Can't Stand - Big Design 2015
cassininazir
355
22k
A better future with KSS
kneath
230
16k
ReactJS: Keep Simple. Everything can be a component!
pedronauck
657
120k
Git: the NoSQL Database
bkeepers
PRO
421
63k
Raft: Consensus for Rubyists
vanstee
130
6.2k
Visualization
eitanlees
135
14k
4 Signs Your Business is Dying
shpigford
174
21k
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