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
コードレビューを通す技術 / technique to pass code review
Search
oohira
April 23, 2019
Programming
1
84
コードレビューを通す技術 / technique to pass code review
oohira
April 23, 2019
Tweet
Share
More Decks by oohira
See All by oohira
git switch & git restore
oohira
0
92
私のエンジニア人生に影響を与えた技術書 / tech books that influenced my engineer life
oohira
1
280
Progressive OSS Contribution
oohira
1
140
Why Mob Programming?
oohira
3
2.1k
朝の15分読書会ふりかえり / 15min-reading-retrospective
oohira
0
150
朝の15分読書会のススメ
oohira
0
420
Ansibleで3台のビルドサーバーを3分で構築した話 / Set up build servers in 3 min by Ansible
oohira
0
170
知っておくと便利なcurl
oohira
0
180
git-svnで始めるボッチのためのGit
oohira
0
4.4k
Other Decks in Programming
See All in Programming
最新TCAキャッチアップ
0si43
0
140
ペアーズにおけるAmazon Bedrockを⽤いた障害対応⽀援 ⽣成AIツールの導⼊事例 @ 20241115配信AWSウェビナー登壇
fukubaka0825
6
1.9k
GitHub Actionsのキャッシュと手を挙げることの大切さとそれに必要なこと
satoshi256kbyte
5
430
Amazon Bedrock Agentsを用いてアプリ開発してみた!
har1101
0
340
subpath importsで始めるモック生活
10tera
0
310
Jakarta EE meets AI
ivargrimstad
0
570
NSOutlineView何もわからん:( 前編 / I Don't Understand About NSOutlineView :( Pt. 1
usagimaru
0
340
Arm移行タイムアタック
qnighy
0
330
初めてDefinitelyTypedにPRを出した話
syumai
0
420
Figma Dev Modeで変わる!Flutterの開発体験
watanave
0
130
Jakarta EE meets AI
ivargrimstad
0
640
RubyLSPのマルチバイト文字対応
notfounds
0
120
Featured
See All Featured
Ruby is Unlike a Banana
tanoku
97
11k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
kevinliebholz
27
4.3k
Scaling GitHub
holman
458
140k
Optimising Largest Contentful Paint
csswizardry
33
2.9k
The Language of Interfaces
destraynor
154
24k
KATA
mclloyd
29
14k
10 Git Anti Patterns You Should be Aware of
lemiorhan
654
59k
What’s in a name? Adding method to the madness
productmarketing
PRO
22
3.1k
Code Reviewing Like a Champion
maltzj
520
39k
Reflections from 52 weeks, 52 projects
jeffersonlam
346
20k
Save Time (by Creating Custom Rails Generators)
garrettdimon
PRO
27
840
Designing for humans not robots
tammielis
250
25k
Transcript
ίʔυϨϏϡʔΛ௨ٕ͢ज़ ϏΞόογϡ-5!PPIJSB
എܠ • ਓΑΓίʔυϨϏϡʔΛࣄͱ͢Δ͕͍࣌ؒ • ͬͱ͜͏ͯ͘͠ΕͨΒΑ͍ͷʹͱ͍͏ࢥ͍ • ϨϏϡʔΞʔͷߟ͍͑ͯΔ͜ͱΛ͑Δ͜ͱͰ ϨϏϡʔΠʔͷॿ͚ʹͳΔͷͰ 2
໔ • ϨϏϡʔΞʔͷҙݟʹݸਓ͕ࠩ͋Γ·͢ • ݱࡏʢ·ͨաڈʣҰॹʹࣄΛ͍ͯ͠Δϝϯ όʔΛ߈ΊΔͷͰ͋Γ·ͤΜ 3
ॳڃฤ
$*Λ௨͢ • ϨϏϡʔΞʔਓؒʹ͔͠Ͱ͖ͳ͍ຊ࣭తͳϨϏϡʔ ׆ಈʹྗ͍ͨ͠ • ελΠϧҧճؼςετػցͷࣄ • ʮ$*͕௨͍ͬͯ·ͤΜʯͱࢦఠ͢Δͷຊస • ͬͯ๙ΊΒΕΔ͜ͱͳ͍͕ɺΒͳ͍ͱϨϏϡʔ
Ξʔʹର͢Δ৴པஷ͕࣮ۚ֬ʹݮΔ 5
$*࣮֬ʹ௨͔ͯ͠Β ϨϏϡʔґཔΛग़ͦ͏ 6
EJGGΛখ͘͞อͭ • ϨϏϡʔΞʔؔͷͳ͍मਖ਼ʹΛ͍ͨ͘ͳ͍ • ͕ࠩখ͚͞Εૉૣ࣮֬͘ʹϨϏϡʔͰ͖Δ • ϘʔΠεΧτϧʔϧͷΓ͗͢ʹҙ ˠࠞͥ͗͢ΔͱͨΓతͳमਖ਼ʹݟ͑ͯෆ҆ʹ • λεΫ.3Ͱ͋Δඞཁͳ͍
7
·ͣࣗͰEJGGΛ֬ೝ͠Α͏ 8
தڃฤ
.3ΛͪΌΜͱॻ͘ • ϨϏϡʔΞʔίʔυͱಉ͙͡Β͍.3ΛϨϏϡʔ͢Δ • .3ʹUZQPίϐϖϛε͕ଟ͍ ˠίʔυͷม໊໊ؔಉ͡ͳͷ͔ͳʁͱෆ҆ʹ • .3ͷઆ໌֬ೝґཔ͕ࡶ ˠ݅ཏ࣮ͯ͠ɾςετ͍ͯ͠Δͷ͔ͳʁͱෆ҆ʹ •
ίʔυ͑ͪ͞ΌΜͱॻ͚ΕΑ͍ͷͰʁ ˠࣗͷ؍ଌൣғͰ.3ΛͪΌΜͱॻ͚Δ͜ͱͱՁ 10
.3ͷ͍͍Ճݮ͞ ίʔυͷ͍͍Ճݮͩ͞ͱ৺ಘ͑Α͏ 11
ࣄલʹ૬ஊ͢Δ • ϨϏϡʔΞʔϓϩδΣΫτશମͷҰ؏ੑʹؔ৺͕ڧ͍ͨ Ίɺ৽͍͠ύλʔϯͷઃܭ࣮ʹ৻ॏʹͳΔ • .3Λग़͔ͯ͠Β૬ஊ͞Εͯɺ͋Ε͜ΕΓ͠ʹ ͳ͓ͬͯޓ͍ͭΒ͔͞͠ͳ͍ • ࣗͰߟ࣮͑ͯ͢Δͷ͕ϓϩάϥϛϯάͷޣຯͩ ͕ɺํͷೝࣝ߹Θͤૣ͍΄ͲखΓΛݮΒͤΔ
• ࣮ͱϨϏϡʔશʹಠཱͨ͠ϑΣʔζͰͳ͍ 12
ૣ͔͘ΒϨϏϡʔΞʔΛר͖ࠐ͏ 13
্ڃฤ
ඍົͳ࣮Λະདྷʹୗ͢ • ͍ͭᘳͳ࣮͕Ͱ͖ΔͱݶΒͳ͍ • ϨϏϡʔΞʔࠓ͑ΔϦιʔεͱকདྷͷίʔυ࣭ͷ όϥϯεΛͱΓ͍ͨ • ͍͚ͯͳ͍࣮ʹݴ͍༁ίϝϯτΛͨ͠Γɺόάཧ γεςϜʹλεΫΛొͨ͠Γ͢Δ
ˠ ՝͕͍ͬͯΔ͜ͱઌૹΓʹͨ͠ཧ༝͕͔Δ 15
ࠓͰ͖ͳͯ͘কདྷͰ͖Δ ֻ͚Λͦ͏ 16
WTࢲͨͪ • ϨϏϡʔΞʔਓؒͳͷͰɺώʔτΞοϓ͢ΔͱϨϏϡʔ ΞʔWTϨϏϡʔΠʔʹͳΓ͕ͪ • ϨϏϡʔखஈɻϨϏϡʔʹ௨͢͜ͱ͕తͰͳ͍ • ϖΞϓϩϞϒϓϩʹΓସ͑ͨํ͕ૣ͘ऴΘΔ߹ •
్த·ͰΛ͍ͬͨΜϚʔδͯ͠͠·ͬͯΓͨ͠ํ ͕ૣ͘ऴΘΔ߹ 17
νʔϜͱͯ͠λεΫΛ ऴΘΒͤΔํ๏Λ͠߹͓͏ 18
൪֎ฤ
པΉ͔Β4UBSUEJTDVTTJPO͍ͬͯͩ͘͞ • ϨϏϡʔΞʔࢦఠΛͭͣͭ࠶֬ೝ͍͖͍ͯͨ͠ • *TTVF.3ͷίϝϯτதʹεϨουΛ࡞ΕΔ 20
·ͱΊ
ͪΐͬͱͷखؒͰ εϜʔζʹϨϏϡʔΛ௨աͤ͞Α͏ 22