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.
→
nayuta24
August 24, 2024
Technology
0
210
コードレビュアー「見習い」 になったぞ!!
フロントエンドカンファレンス北海道でのスポンサーLTです。
nayuta24
August 24, 2024
Tweet
Share
More Decks by nayuta24
See All by nayuta24
グローバル変数は危ないって聞いたけど なんでReduxはいいの??
nayuta24
0
440
Other Decks in Technology
See All in Technology
パワポ作るマンをMCP Apps化してみた
iwamot
PRO
0
220
なぜarray_firstとarray_lastは採用、 array_value_firstとarray_value_lastは 見送りだったか / Why array_value_first and array_value_last was declined, then why array_first and array_last was accpeted?
cocoeyes02
0
270
Embeddings : Symfony AI en pratique
lyrixx
0
410
MCPで決済に楽にする
mu7889yoon
0
160
韓非子に学ぶAI活用術
tomfook
4
1.2k
VSCode中心だった自分がターミナル沼に入門した話
sanogemaru
0
840
Oracle Cloud Infrastructure(OCI):Onboarding Session(はじめてのOCI/Oracle Supportご利⽤ガイド)
oracle4engineer
PRO
2
17k
Datadog で実現するセキュリティ対策 ~オブザーバビリティとセキュリティを 一緒にやると何がいいのか~
a2ush
0
170
スケールアップ企業でQA組織が機能し続けるための組織設計と仕組み〜ボトムアップとトップダウンを両輪としたアプローチ〜
qa
0
380
Oracle AI Database@AWS:サービス概要のご紹介
oracle4engineer
PRO
3
2k
AWS Systems Managerのハイブリッドアクティベーションを使用したガバメントクラウド環境の統合管理
toru_kubota
1
190
QA組織のAI戦略とAIテスト設計システムAITASの実践
sansantech
PRO
1
250
Featured
See All Featured
The browser strikes back
jonoalderson
0
850
Mozcon NYC 2025: Stop Losing SEO Traffic
samtorres
0
190
Stop Working from a Prison Cell
hatefulcrawdad
274
21k
The Illustrated Children's Guide to Kubernetes
chrisshort
51
52k
Balancing Empowerment & Direction
lara
5
1k
Design of three-dimensional binary manipulators for pick-and-place task avoiding obstacles (IECON2024)
konakalab
0
390
Large-scale JavaScript Application Architecture
addyosmani
515
110k
My Coaching Mixtape
mlcsv
0
87
世界の人気アプリ100個を分析して見えたペイウォール設計の心得
akihiro_kokubo
PRO
68
38k
The Anti-SEO Checklist Checklist. Pubcon Cyber Week
ryanjones
0
100
Data-driven link building: lessons from a $708K investment (BrightonSEO talk)
szymonslowik
1
990
GitHub's CSS Performance
jonrohan
1032
470k
Transcript
コードレビュアー「見習い」 になったぞ!! 株式会社カオナビ 中村優太
自己紹介 中村優太 名前: 2000/08/24 生年月日: 株式会社カオナビ(23卒) 所属: 音楽、あんこをつくること 好きなこと: @tanasokoo
X:
None
コードレビュー 最高ですよね!!😄 (唐突な圧力)
コードレビュー なんのためにやる?
参考 参考 https://speakerdeck.com/utgwkk/da-ji-xiang-si-dot-pm
参考 参考 https://speakerdeck.com/utgwkk/da-ji-xiang-si-dot-pm
コードの品質を保つため 知見を共有するため
メリットはたくさん
そうはいっても
レビューするのは
怖い!!
特に若手!!!
なにが怖い??
先輩エンジニアへ 適切なレビュー・質問ができるか不安
• 命名が微妙かも? • このas(TypeScript)って必要なんですか? • Reduxのactionがeventではなくsetterっぽくなっ てません?
そもそも指摘できるところ が全く見つけられないかも
ましてやLGTMなんて 絶対無理!!!!
None
• コードレビュアー「試用期間」制度とは • レビュアーになってわかったこと、よかったこと • レビュアー1stステップ ながれ
若手が抱える レビューへの恐怖
そもそも何やってるのか まったく理解できないかも
LGTMを押すほど自信がない
でもいろいろ 勉強になるって聞くし...
やってみたくはある...
そんな願いを叶えるのが
レビュアー試用期間制度
• レビューマスターと呼ばれるレビュアーがいる • レビューマスターになるために ◦ 「試用期間」制度が1stステップ 弊社のレビュアー制度
• 通常のレビュアーと同様にLGTMが出せる • よくわからなかったらレビューマスターへのパスOK! • 3ヶ月に1回面談、フィードバックを受けて昇格 or 継続 レビュアー試用期間制度
比較的低いハードルで レビュー経験を重ねられる
レビュアー見習いに なってみて
• やはり勉強になる • レビュアーをリスペクトできるように ◦ 見やすいプルリクを作る • セルフレビューできるようになる 若手がレビュアー見習いになってみて
レビュアー見習いに なりたいと思ったあなた
若手をレビュアー見習い にしたいと思ったあなた
でも...
何をどうレビューしたら いいか分からん💢💢💢
そんなあなたに
レビュアー 1stステップ
• わからないところは、わかりませんと素直に言う • 「ここいいですね!勉強になります!!」だけでもOK! • ChatGPTに投げてみる レビュアー1stステップ ※ 社内ルールに従ってください
これであなたも 明日からレビュアーに🤘
コードレビュアー「見習い」 になったぞ!! 株式会社カオナビ 中村優太
おわり!!!
の前に...
もう一つだけ
None
お分かりいただけただろうか...
とても重大なことが書かれていた
自己紹介 中村優太 名前: 2000/08/24 生年月日: 株式会社カオナビ 所属: 音楽、あんこをつくること 好きなこと: @tanasokoo
X:
自己紹介 中村優太 名前: 2000/08/24 生年月日: 株式会社カオナビ 所属: 音楽、あんこをつくること 好きなこと: @tanasokoo
X:
今日誕生日です
会場にいます
懇親会にもいます
そこのあなた
プレゼントください
は、無理だと思うので...
たくさん話しかけてください