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
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
430
Other Decks in Technology
See All in Technology
マネージャー視点で考えるプロダクトエンジニアの評価 / Evaluating Product Engineers from a Manager's Perspective
hiro_torii
0
180
今こそ学びたいKubernetesネットワーク ~CNIが繋ぐNWとプラットフォームの「フラッと」な対話
logica0419
5
450
顧客との商談議事録をみんなで読んで顧客解像度を上げよう
shibayu36
0
310
予期せぬコストの急増を障害のように扱う――「コスト版ポストモーテム」の導入とその後の改善
muziyoshiz
1
2.1k
生成AIを活用した音声文字起こしシステムの2つの構築パターンについて
miu_crescent
PRO
3
220
フルカイテン株式会社 エンジニア向け採用資料
fullkaiten
0
10k
We Built for Predictability; The Workloads Didn’t Care
stahnma
0
150
SREが向き合う大規模リアーキテクチャ 〜信頼性とアジリティの両立〜
zepprix
0
480
配列に見る bash と zsh の違い
kazzpapa3
3
170
20260204_Midosuji_Tech
takuyay0ne
1
160
Agent Skils
dip_tech
PRO
0
130
こんなところでも(地味に)活躍するImage Modeさんを知ってるかい?- Image Mode for OpenShift -
tsukaman
1
170
Featured
See All Featured
Mobile First: as difficult as doing things right
swwweet
225
10k
Navigating Weather and Climate Data
rabernat
0
110
Chasing Engaging Ingredients in Design
codingconduct
0
110
jQuery: Nuts, Bolts and Bling
dougneiner
65
8.4k
Accessibility Awareness
sabderemane
0
56
Skip the Path - Find Your Career Trail
mkilby
0
59
Documentation Writing (for coders)
carmenintech
77
5.3k
brightonSEO & MeasureFest 2025 - Christian Goodrich - Winning strategies for Black Friday CRO & PPC
cargoodrich
3
110
Bash Introduction
62gerente
615
210k
The Spectacular Lies of Maps
axbom
PRO
1
530
Believing is Seeing
oripsolob
1
58
職位にかかわらず全員がリーダーシップを発揮するチーム作り / Building a team where everyone can demonstrate leadership regardless of position
madoxten
58
50k
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:
今日誕生日です
会場にいます
懇親会にもいます
そこのあなた
プレゼントください
は、無理だと思うので...
たくさん話しかけてください