Lock in $30 Savings on PRO—Offer Ends Soon! ⏳
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
コードレビュアー「見習い」 になったぞ!!
Search
nayuta24
August 24, 2024
Technology
0
200
コードレビュアー「見習い」 になったぞ!!
フロントエンドカンファレンス北海道でのスポンサーLTです。
nayuta24
August 24, 2024
Tweet
Share
More Decks by nayuta24
See All by nayuta24
グローバル変数は危ないって聞いたけど なんでReduxはいいの??
nayuta24
0
410
Other Decks in Technology
See All in Technology
1人1サービス開発しているチームでのClaudeCodeの使い方
noayaoshiro
1
180
業務のトイルをバスターせよ 〜AI時代の生存戦略〜
staka121
PRO
2
200
マイクロサービスへの5年間 ぶっちゃけ何をしてどうなったか
joker1007
6
1.9k
エンジニアリングマネージャー はじめての目標設定と評価
halkt
0
290
5分で知るMicrosoft Ignite
taiponrock
PRO
0
380
AWS Security Agentの紹介/introducing-aws-security-agent
tomoki10
0
280
Sansanが実践する Platform EngineeringとSREの協創
sansantech
PRO
2
880
Fashion×AI「似合う」を届けるためのWEARのAI戦略
zozotech
PRO
2
670
Lambdaの常識はどう変わる?!re:Invent 2025 before after
iwatatomoya
1
560
EM歴1年10ヶ月のぼくがぶち当たった苦悩とこれからへ向けて
maaaato
0
280
AIプラットフォームにおけるMLflowの利用について
lycorptech_jp
PRO
1
160
新 Security HubがついにGA!仕組みや料金を深堀り #AWSreInvent #regrowth / AWS Security Hub Advanced GA
masahirokawahara
1
2.1k
Featured
See All Featured
Being A Developer After 40
akosma
91
590k
[Rails World 2023 - Day 1 Closing Keynote] - The Magic of Rails
eileencodes
37
2.6k
Product Roadmaps are Hard
iamctodd
PRO
55
12k
Rebuilding a faster, lazier Slack
samanthasiow
84
9.3k
Testing 201, or: Great Expectations
jmmastey
46
7.8k
VelocityConf: Rendering Performance Case Studies
addyosmani
333
24k
Six Lessons from altMBA
skipperchong
29
4.1k
Mobile First: as difficult as doing things right
swwweet
225
10k
The Language of Interfaces
destraynor
162
25k
The Web Performance Landscape in 2024 [PerfNow 2024]
tammyeverts
12
970
For a Future-Friendly Web
brad_frost
180
10k
Build The Right Thing And Hit Your Dates
maggiecrowley
38
3k
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:
今日誕生日です
会場にいます
懇親会にもいます
そこのあなた
プレゼントください
は、無理だと思うので...
たくさん話しかけてください