コードレビューの文化を手探りで作っていった話

#code_kaizen
コード改善 #2 で発表したスライドです。

Transcript

1. ίʔυϨϏϡʔͷจԽΛ
 ख୳Γ࡞͍ͬͯͬͨ࿩ Yasuhiro Kiyota @yasuhiroki
2. ࣗݾ঺հ
3. ࣗݾ঺հ • Yasuhiro Kiyota • @ysauhiroki • Twitter: @duck_yasuhiroki • GitHub: https://github.com/yasuhiroki • ։ൃऀʢݸਓʣͷͨΊͷJenkins - Git Pluginฤ • http://qiita.com/yasuhiroki/items/61a2be613fc7dcfc8682 • ͦͷγΣϧεΫϦϓτ΋͏ͪΐͬͱγϯϓϧʹॻ͚ͦ͏ TipsूʢGolf/γΣϧܳͰ͸ͳ͍ʣ • http://qiita.com/yasuhiroki/items/fad876db9e5505fceb03
4. https://certificates.cloudbees.com/10203903
5. झຯ $ echo "#code_kaizen" | grep -o . | sed 1d | tr '_' '|' | gsed -e '1i+-+' -e 's/./|&|/' -e '$a+-+'
6. ຊ୊
7. ίʔυϨϏϡʔͷจԽΛ
 ख୳Γ࡞͍ͬͯͬͨ࿩
8. Agenda • ࢲͱ৽ଔͷ2ਓνʔϜͷ࣌ • ࢲͱ2೥໨2ਓͷ3ਓνʔϜͷ࣌
9. ࢲͱ৽ଔͷ2ਓνʔϜͷ࣌
10. ౰࣌ͷঢ়گ • ৽نαʔϏε։ൃ • ϓϩτλΠϓ։ൃ • ΫϥΠΞϯτ • AndroidΞϓϦ • αʔόʔαΠυ • AWS RDS + AWS Lambda + AWS API Gateway ͷ
 αʔόʔϨεߏ੒ʹॳ௅ઓ
11. ౰࣌ͷঢ়گ • νʔϜߏ੒ • ϓϩδΣΫτϦʔμʔ • ࢲ • ৽ଔ • ઌഐʢAndroidͷ͝ҙݟ൛ʣ • ։ൃ͸αʔόʔαΠυ͕த࣠ • αʔόʔϨεॳ௅ઓ • αʔόʔαΠυ։ൃ΋ɺνʔϜͱͯ͠͸ॳ௅ઓ
12. ౰࣌ͷঢ়گ • ༗ࣝऀෆࡏ
13. ౰࣌ͷঢ়گ • ༗ࣝऀෆࡏ • νʔϜͰ։ൃͨ͠ܦݧ΋গͳ͍
14. ౰࣌ͷঢ়گ • ༗ࣝऀෆࡏ • νʔϜͰ։ൃͨ͠ܦݧ΋গͳ͍
 ɹ˞ ։ൃɾӡ༻πʔϧ։ൃνʔϜͩͬͨ
 ɹ˞ 1ਓͣͭผʑͷπʔϧΛ࡞ͬͯΔײ͡
15. ౰࣌ͷঢ়گ • ༗ࣝऀෆࡏ • νʔϜͰ։ൃͨ͠ܦݧ΋গͳ͍ ← νʔϜ։ൃ΁ͷಌΕ
 ɹ˞ ։ൃɾӡ༻πʔϧ։ൃνʔϜͩͬͨ
 ɹ˞ 1ਓͣͭผʑͷπʔϧΛ࡞ͬͯΔײ͡
16. ࢲͷࢥ͍
17. ౰࣌ͷࢲͷࢥ͍ • ։ൃऀͱͯ͠ • ૣ͘ྑ͍੒ՌΛग़͢ʹ͸Ͳ͏͢Ε͹ྑ͍ͷ͔ • ྑ͍ίʔυΛॻ͘ʹ͸Ͳ͏͢Ε͹ྑ͍ͷ͔ • ઌഐͱͯ͠ • ҆৺ͯ͠։ൃͯ͠΋Β͏ʹ͸Ͳ͏͢Ε͹ྑ͍ͷ͔ • ࢲͷѱ͍ίʔυΛѱ͍ͱݴͬͯ΋Β͏ʹ͸Ͳ͏͢Ε͹ ྑ͍ͷ͔
18. ߟ͑ͯΈͨ
19. ͜Ε͕Ͱ͖Ε͹ྑ͍ͷͰ͸ʁ • ຖேϓνձΛ։͘ • ຖ೔ͪΌΜͱλεΫͷ੔ཧͱεϓϦϯτͷ໨ඪ֬ೝ Λ͢Δ • ࢥͬͨ͜ͱ͸ԿͰ΋ݴ͍߹͍͍ͨͷͰɺؾܰ͞Λҙ ࣝͯ͠ϓνձͱݺশ • ޓ͍ʹԕྀͳ͘ίʔυϨϏϡʔΛ͢Δ • ίʔυϨϏϡʔͷϧʔϧʢ৺ߏ͑ʣΛ࡞Δ • ·ͣ͸ܗ͔Β
20. ࣮ફ
21. ίʔυϨϏϡʔͷϧʔϧ • ϨϏϡʔϫʔͷ৺ߏ͑ • ϓϧϦΫ͕ग़͞Εͨ࣍ͷ೔ʹ͸ϨϏϡʔ͢Δ • ίʔυ্Ͱ͸ઌഐޙഐ͸ؔ܎ͳ͍ • ϨϏϡʔΠͷ৺ߏ͑ • SlackͰϓϧϦΫग़ͨ͜͠ͱΛඞͣ࿈བྷ͢Δ • ͋͘·Ͱ͓ئ͍͢Δ͜ͱ
22. ίʔυϨϏϡʔґཔ࣌ͷςϯϓϨʔτ • .github/PULL_REQUEST_TEMPLATE.md Λ༻ҙ • ֓ཁ • ͦͷϓϧϦΫʹΑͬͯԿ͕ୡ੒͞Ε͔ͨ • ࣮૷಺༰ʢνΣοΫϘοΫεʣ • ࣮૷ͨ͠಺༰ͷՕ৚ॻ͖ • ݒ೦఺ • ≒ ݴ͍༁Λؾ࣋ͪΛॻ͘ • ʮ͜͜͸Զ΋ݏͳΜ΍…ʯ • ʮ࢓༷͕ܾ·ͬͨΒϦϑΝΫλ͢Δ͔Β…ʯ • Ϛʔδͯ͠Α͠ • ݟ·ͨ͠ΑνΣοΫϘοΫε
23. ίʔυϨϏϡʔґཔ࣌ͷςϯϓϨʔτͷޮՌ • ϨϏϡʔϫʔࢹ఺ • ݒ೦఺ʹ͍ͭͯࢦఠ͕Ͱ͖Δ • ʮPMʹ֬ೝऔ͔ͬͨΒ͜ΕͰྑ͍Α or ௚ͯ͠ʯ • ϨϏϡʔΠࢹ఺ • ґཔલʹݟ௚͕͠Ͱ͖Δ • ʮ࣮૷಺༰ͷՕ৚ॻ͖͕ଟ͗͢Δ͔ΒผͷϓϧϦΫ ʹ෼͚Α͏ʯ
24. ίʔυϨϏϡʔґཔ࣌ͷϥϕϧ • WIP / PleaseReviewϥϕϧΛ༻ҙ
25. ίʔυϨϏϡʔґཔ࣌ͷϥϕϧͷޮՌ • ϨϏϡʔΛґཔ͠Α͏ͱࢥ͍ͬͯͨΒɺ
 ͢ͰʹϨϏϡʔ͕ऴΘ͍ͬͯͨ • ʮPleaseReviewϥϕϧ͕షͬͯ͋ͬͨͷͰɺ
 ɹϨϏϡʔͯ͠ίϝϯτॻ͍ͯஔ͖·ͨ͠ʯ
26. औΓ૊Έͷ݁Ռ • 43 / 51 (68%) ͷϓϧϦΫΤετ͕
 ࡞੒͔Β1~3೔Ҏ಺ʹ Merge or Close
 ɹ※ Please Review ϥϕϧ͕షΒΕ͔ͯΒͰ͸ͳ͍
 ɹɹͦͷ৔߹ɺ΋ͬͱ૿͑Δ૝ఆ (ूܭͰ͖ͯͳ͍) • ͭ·ΓϓϧϦΫޙͷ
 ϨϏϡʔ -> ࢦఠՕॴͷमਖ਼ -> ࠶ϨϏϡʔ -> Ϛʔδ ͕ɺ
 1~3೔ޙʹ͸ɺ͍͍ͩͨ׬͍ྃͯ͠Δ
27. ͏·͘ػೳͯͦ͠͏ʢʁʣ
28. ࢲͱ2೥໨2ਓͷ3ਓνʔϜͷ࣌
29. ౰࣌ͷঢ়گ • ϓϩδΣΫτͷن໛͕ͪΐͬͱ֦େ • AndroidΞϓϦ • ຊϦϦʔεʹ޲͚ͯ௥͍ࠐΈ • ͋ͱผΞϓϦΛɺ΋͏̍ͭ௥ՃͰʂ • αʔόʔαΠυͷ࣮૷͸ޙճ͠ͷϓϩτλΠϓΛ • αʔόʔαΠυ • ຊϦϦʔεʹ޲͚ͯ௥͍ࠐΈ
30. ౰࣌ͷঢ়گ • νʔϜߏ੒ • ϓϩδΣΫτϚωʔδϟʔ • αϒϦʔμʔ • ࢲ • 2೥໨ (લճͷ৽ଔ) • 2೥໨ (New) • ઌഐʢ͝ҙݟ൪ʣ • ผΞϓϦ୲౰
31. ౰࣌ͷঢ়گ • Ұॹʹ΍͖ͬͯͨ৽ଔ͕2೥໨ʹʂ • ৽ͨʹ 2೥໨΋ՃΘͬͯ 3ਓମ੍ʹʂ • νʔϜશମͰ΋ਓһ͕૿͑ͨʂ • AndroidΞϓϦ։ൃ͕த࣠ʹ
32. ࢲͷࢥ͍
33. ౰࣌ͷࢲͷࢥ͍ • ҎલͷऔΓ૊ΈΛܧଓͯ͠ଓ͚͍ͨ
34. ౰࣌ͷࢲͷࢥ͍ • ҎલͷऔΓ૊ΈΛܧଓͯ͠ଓ͚͍ͨ
35. ౰࣌ͷࢲͷࢥ͍ • ҎલͷऔΓ૊ΈΛܧଓͯ͠ଓ͚͍ͨ
36. ౰࣌ͷࢲͷࢥ͍ • ҎલͷऔΓ૊ΈΛܧଓͯ͠ଓ͚͍ͨ
37. ౰࣌ͷࢲͷࢥ͍ • ҎલͷऔΓ૊ΈΛܧଓͯ͠ଓ͚͍ͨ • ͦΕ͚ͩ
38. Կ͕ى͖͔ͨ
39. ൃੜͨ͠໰୊ 1 • ίϛϡχέʔγϣϯྔͷ૿େ • ̍ਓ૿͑ͨ͸ɺ+1 Ͱ͸ͳ͘ *2 ͩͬͨ • ίʔυϨϏϡʔґཔͷ૿Ճ • ࢲʮ͝ΊΜɺϨϏϡʔ໌೔Ҏ߱ʹͳΓ·͢ʯ͕ଟൃ • ࢲʮ͝ҙݟ൪͕OKͬͯݴͬͨΒOKͰ͢ʯ • ͝ҙݟ൪ʮ࢓༷͕෼͔Βͳ͍ͷͰࡉ͔͘ݟΕͯͳ͍ ͚Ͳɺίί͕nullͷ࣌ͷৼΔ෣͍͕͜ΕͰྑ͍ͳΒ OKͰ͢ʯ
40. ൃੜͨ͠໰୊ 2 • 2೥໨(NEW) ͱͷจԽతɾٕज़తഎܠͷࠩ • ࢲ͕ͨͪഓ͖ͬͯͨϧʔϧɾٕज़Λઆ໌ͤͣਐΊ͍ͯͨ • ࢲʮ෼͔Μͳ͍͜ͱ͋ͬͨΒฉ͍ͯʔʯ • ʮϨϏϡʔ͸ԿΛݟΕ͹…ʯ • -> ίʔυϨϏϡʔͷจԽ͕ͳ͔ͬͨ • ʮGitͷ࢖͍ํͬͯ͜ΕͰྑ͍ͷͰ͠ΐ͏͔…ʯ • -> ͦΕ·ͰSCM͕ෆཁͳ։ൃ͕ओͩͬͨ • ʮࢼ͠ʹ࣮૷͚ͨͩ͠ͳͷͰ push ͯ͠ͳ͍Ͱ͢ʯ • -> ਖ਼͘͠ಈ͘΋ͷ͚ͩΛ remote ʹ͋͛ΔจԽͩͬͨ
41. ໰୊ͷ૯ׅ • ৽͘͠དྷͨਓΛड͚ೖΕΔମ੍͕શ͘੔ͬͯͳ͍ • ࣭໰͞Εͳ͍ͱೝࣝҧ͍ʹؾ͚ͮͳ͍ • ࢲ͕ϘτϧωοΫʹͳ͍ͬͯΔ • ͦΕ·Ͱͱಉ͡։ൃϖʔεΛҡ࣋ͯ͠৽͍͠։ൃʹ΋ टΛಥͬࠐΜͰ͍Δ • ܻ͋;Εঢ়ଶ… • ੹೚ͷؙ౤͛ͱ͍͏࠷௿ͷߦҝ…
42. ͜ͷ··Ͱ͸μϝͩͱؾ෇͘
43. ߟ͑ͯΈͨ
44. ॳ৺ʹؼΔ
45. ΍ͬͨ͜ͱ
46. ͱʹ͔͘࿩͢ • ·ͣ͸ޱ಄Ͱɺͱʹ͔͘ձ࿩͢Δ • ʮࢲͷϓϧϦΫͷɺ͜ͷลͲ͏ࢥ͏ʁʯ • ʮͺͬͱݟͨײ͡ɺ৚݅෼ذ͕෼͔ΓͮΒ͍Ͱ͢ʯ • ʮ͓͚ɺϝιουԽ͢ΔΘʯ • ࣗ෼ͷ࡞ۀΛࢭΊͯͰ΋ɺձ࿩Λ࠷༏ઌʹ • Slack΋׆༻ • ͱʹ͔͘࿩͔͚͠Δ • ͓ޓ͍ʹ׳ΕΔ
47. ׳Εͯ΋Β͏ • ձ࿩ͷ಺༰ΛGitHubʹ΋ॻ͍ͯ΋Β͏ • ʮೋ౓ख͚ؒͩͲɺ
 ɹࠓ࿩ͨ͠಺༰ΛGitHubʹ΋ॻ͍ͯ΋Β͍͍ͬͯʁ
 ɹͦͨ͠Β௚͠·ͨͬͯ͠ίϝϯτฦ͢ΜͰʯ • औֻ͔ͬΓ͕Ͱ͖Ε͹ɺ͋ͱ͸εϜʔζʹਐΜͩɻ
48. ࢲࣗ਎ͷλεΫͷ੔ཧ • ਎ͷৎʹԊͬͨλεΫྔʹͳΔΑ͏ௐ੔͢Δ • CIͱ͔σϓϩΠࣗಈԽͱ͔ޙճ͠ʹ
 ˞ ͦΕ͸ͦΕͰ໰୊ͳؾ΋͢Δ͕… • গͣͭ͠2೥໨ʹࢲͷλεΫΛҕৡ͢Δ • ͝ҙݟ൪ͱɺͪΌΜͱωΰΔ • ࢲʮ࣮ػͰͷಈ࡞֬ೝ͸ͨ͠ͷͰ
 ɹɹίʔυϨϏϡʔΛ͓ئ͍͠·͢ʯ
49. ૯ͯ͡ݴ͑Δ͜ͱ • ࿩͢ͷ͕Ұ൪ • ΍ͬͯΈͤɺ͍͖͔ͬͯͤɺͤͯ͞Έͤ • ͳ͋ͳ͋ɺʹ ͳΒͳ͍Α͏ؾΛ͚ͭΔ
50. ੒Ռ • 2೥໨(NEW) ͕த৺ͷ৽ΞϓϦͷPR͸ɺ
 49/61(80%) ͕ 1೔ޙ·ͰʹϚʔδ͞Εͨ • 1݅ɺ Ϛʔδ·Ͱ30೔͔͔ͬͨϓϧϦΫ͕͋Δ • Ұ൪ॳΊͷϓϧϦΫ • Gitͷ࢖͍ํɺ։ൃํ਑ͷڞ༗͕Ͱ͖͍ͯͳ͔ͬͨͨ Ί • ͜ͷ1݅Ҏ߱ɺૣ͍ϓϧϦΫͷαΠΫϧ͕࣮ݱͰ͖ͨ
51. ੒Ռ • 2೥໨(લճ৽ଔ) ͕த৺ͷΞϓϦͷPR͸ɺ
 80/107(74%) ͕ 1೔ޙ·ͰʹϚʔδ͞Εͨ
52. ੒Ռ • ৄࡉʹܭଌ͍ͯ͠ͳ͍͕ɺ
 ϨϏϡʔίϝϯτ਺΋͕࣌ؒܦͭʹͭΕͯ૿͍͑ͯͬͨ
53. ࠓޙͷ՝୊
54. ࠓޙͷ՝୊ • ίʔυϨϏϡʔͷҙࣝ͸ߴ·͍ͬͯΔ͕… • Ռͨͯ͠ίʔυ͸ྑ͘ͳ͍ͬͯΔͷ͔Ṗ • Կ͔ࢦඪΛઃ͚Δʁ • ΋ͬͱϨϏϡʔͷෛՙΛԼ͛ΒΕΔͷͰ͸ • ϨϏϡʔ༻ͷνΣοΫγʔτΛ༻ҙ͢Δʁ • ΋ͬͱπʔϧΛ࢖͏ʁ • ॳ৺ ๨Ε͕ͪ
55. ͓͠·͍