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
古い指輪から広がった JAVA コミュニティ とのつながり
yamad
0
130
パスキーの自動テストはどこまでいけるのか
teamlab
PRO
0
290
NDIでApple Vision Proにリアルタイム映像送信
jyouryuusui
0
130
Browser Spec: Testing Hotwire with Ruby
takahashim
1
130
hackers_champloo_2026_tidb
makocchi
0
170
「Docker Desktop不要の時代」は 来たのか?
ueponx
0
840
日本語に強いオープンな大規模言語モデル「LLM-jp」
odashi
5
1.2k
JAWSUG TOHOKU 青森支部 「LoGoチャット」のAWS移行で 乗り越えたこと
taketakekaho
0
350
『SAR衛星データ解析入門』のご紹介とその応用
syuchimu
0
120
会社紹介資料 / Sansan Company Profile
sansan33
PRO
31
430k
AI駆動で「他アプリの音を加工できる」アプリを作った
satomasahiro2005
0
170
Code for Security プロジェクト紹介 2026年10月
yu23ki14
0
110
Featured
See All Featured
Beyond borders and beyond the search box: How to win the global "messy middle" with AI-driven SEO
davidcarrasco
3
280
Writing Fast Ruby
sferik
630
63k
10 Git Anti Patterns You Should be Aware of
lemiorhan
PRO
659
62k
From π to Pie charts
rasagy
1
400
<Decoding/> the Language of Devs - We Love SEO 2024
nikkihalliwell
1
360
HDC tutorial
michielstock
2
960
Information Architects: The Missing Link in Design Systems
soysaucechin
1
1.2k
Lessons Learnt from Crawling 1000+ Websites
charlesmeaden
PRO
1
1.6k
SERP Conf. Vienna - Web Accessibility: Optimizing for Inclusivity and SEO
sarafernandez
2
1.6k
AI: The stuff that nobody shows you
jnunemaker
PRO
10
1.2k
世界の人気アプリ100個を分析して見えたペイウォール設計の心得
akihiro_kokubo
PRO
74
43k
Code Reviewing Like a Champion
maltzj
529
40k
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:
今日誕生日です
会場にいます
懇親会にもいます
そこのあなた
プレゼントください
は、無理だと思うので...
たくさん話しかけてください