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
430
承認を意識したコードレビュー
コードレビュー LT会 - vol.3 2022/10/12
buntafujikawa
December 02, 2022
Tweet
Share
More Decks by buntafujikawa
See All by buntafujikawa
スクラムとコーチングで作る心理的安全性
buntafujikawa
0
610
ちょっと便利になる自動補完と Ansible でのパッケージ管理について
buntafujikawa
0
680
社内技術広報のすすめ
buntafujikawa
0
150
テストコードについて (仮)
buntafujikawa
0
530
テトリスでイメージする 技術的負債とリファクタリング
buntafujikawa
0
880
設計レビューのすすめ
buntafujikawa
1
550
Write Code Every Day
buntafujikawa
0
290
ドラゴン桜から学んだ勉強法
buntafujikawa
1
4.5k
How to do effective pair programming with junior engineers
buntafujikawa
1
650
Other Decks in Technology
See All in Technology
私なりのAIのご紹介 [2024年版]
qt_luigi
1
120
20241218_今年はSLI/SLOの導入を頑張ってました!
zepprix
0
100
3年でバックエンドエンジニアが5倍に増えても破綻しなかったアーキテクチャ そして、これから / Software architecture that scales even with a 5x increase in backend engineers in 3 years
euglena1215
9
3.7k
非機能品質を作り込むための実践アーキテクチャ
knih
5
1.6k
Work as an App Engineer
lycorp_recruit_jp
0
370
20241220_S3 tablesの使い方を検証してみた
handy
4
700
新機能VPCリソースエンドポイント機能検証から得られた考察
duelist2020jp
0
230
サイバー攻撃を想定したセキュリティガイドライン 策定とASM及びCNAPPの活用方法
syoshie
3
1.4k
.NET 9 のパフォーマンス改善
nenonaninu
0
1.3k
成果を出しながら成長する、アウトプット駆動のキャッチアップ術 / Output-driven catch-up techniques to grow while producing results
aiandrox
0
390
[トレノケ雲の会 mod.13] 3回目のre:Inventで気づいたこと -CloudOperationsを添えて-
shintaro_fukatsu
0
110
AWS re:Invent 2024 ふりかえり勉強会
yhana
0
480
Featured
See All Featured
Building a Modern Day E-commerce SEO Strategy
aleyda
38
7k
Embracing the Ebb and Flow
colly
84
4.5k
Writing Fast Ruby
sferik
628
61k
JavaScript: Past, Present, and Future - NDC Porto 2020
reverentgeek
47
5.1k
For a Future-Friendly Web
brad_frost
175
9.4k
Designing Experiences People Love
moore
138
23k
YesSQL, Process and Tooling at Scale
rocio
170
14k
The Illustrated Children's Guide to Kubernetes
chrisshort
48
48k
Reflections from 52 weeks, 52 projects
jeffersonlam
347
20k
GraphQLの誤解/rethinking-graphql
sonatard
67
10k
The Invisible Side of Design
smashingmag
298
50k
Git: the NoSQL Database
bkeepers
PRO
427
64k
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