Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Sign up for free
Menu
Search
Features
All features
Private URLs
Password Protection
Custom URLS
Scheduled publishing
Remove Branding
Restrict embedding
Deck Collections
Notes
Features
All features
Private URLs
Password Protection
Custom URLS
Scheduled publishing
Remove Branding
Restrict embedding
Deck Collections
Notes
Explore
Featured decks
Featured speakers
Programming
Technology
Storyboards
Explore
Featured decks
Featured speakers
Programming
Technology
Storyboards
Pricing
Search
Sign in
Sign up for free
コードレビュアー「見習い」 になったぞ!!
Search
nayuta24
August 24, 2024
Technology
250
0
Share
Embed
Copy iframe code
Copy JS code
Copy link
Start on current slide
コードレビュアー「見習い」 になったぞ!!
フロントエンドカンファレンス北海道でのスポンサーLTです。
nayuta24
August 24, 2024
More Decks by nayuta24
See All by nayuta24
グローバル変数は危ないって聞いたけど なんでReduxはいいの??
nayuta24
0
470
Other Decks in Technology
See All in Technology
Cloudflare Workers 向けアプリを C# で構築する ~WASM Native AOT への道~
nenonaninu
1
1.7k
Coil3を内部実装から読み解く~キャッシュ戦略とAVIF画像の描画〜/nikkei-tech-talk50
nikkei_engineer_recruiting
0
110
[Kiro Meetup #7] Kiro Crew Dive Deep
konippi
0
370
作って終わりじゃないサーバーレス 〜9年運用する大規模EC物流API基盤の設計・運用のリアル〜
zozotech
PRO
0
550
猫でもわかるKiro Web
kentapapa
1
120
AIエージェントの権限管理 3: Agentic RAG の Fine grained access control 編
ren8k
1
280
spanner-autoscalerに学ぶ CRD設計パターン 〜自動化と緊急時対応を両立する Kubernetesコントローラーの作り方〜
tkuchiki
0
180
Apache Iceberg が拓く AI 時代のオープンレイクハウス
tomtanaka
0
110
2026-09-08 そのJavaモダナイゼーション、AIに丸投げで大丈夫?IBM Bobで変わる品質と効率
yutanonaka
1
200
3人で1000GPU超を統合運用する?マルチクラウド&オンプレを跨ぐ、構築と運用のリアル!
kazukun0716
2
500
あけおめLINE 傾向とその対策
nasa9084
0
340
ScotSecure West 2026 - Glasgow
raybugg
0
190
Featured
See All Featured
Thoughts on Productivity
jonyablonski
76
5.4k
Agile Leadership in an Agile Organization
kimpetersen
PRO
0
250
Large-scale JavaScript Application Architecture
addyosmani
515
110k
What Being in a Rock Band Can Teach Us About Real World SEO
427marketing
0
1.1k
Building AI with AI
inesmontani
PRO
1
1.3k
Stop Working from a Prison Cell
hatefulcrawdad
274
21k
How To Stay Up To Date on Web Technology
chriscoyier
790
250k
Visualization
eitanlees
153
17k
Refactoring Trust on Your Teams (GOTO; Chicago 2020)
rmw
35
3.8k
So, you think you're a good person
axbom
PRO
2
2.2k
The Curious Case for Waylosing
cassininazir
1
540
The Art of Programming - Codeland 2020
erikaheidi
57
14k
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:
今日誕生日です
会場にいます
懇親会にもいます
そこのあなた
プレゼントください
は、無理だと思うので...
たくさん話しかけてください