レビューをもらいやすい細かい プルリクの切り分け方 at PHPerKaigi 2018

PHPerKaigi 2018での発表資料です。
http://khigashigashi.hatenablog.com/entry/2018/03/09/020359

Transcript

1. 1)1FS,BJHJ
   
    #"4&גࣜձࣾ
   
   ౦ޱ࿨ᏻʢ!,IJHBTIJHVDIJʣ
   ϨϏϡʔΛ΋Β͍΍͍͢ࡉ͔͍
   ϓϧϦΫͷ੾Γ෼͚ํ
2. ࣗݾ঺հ !,IJHBTIJHVDIJ
   ౦ޱ࿨ᏻ w ॴଐɿ#"4&גࣜձࣾ
   
   #"4&
   1SPEVDU
   %JWJTJPO
   
   αʔόαΠυΤϯδχΞ
   1)1FSઈࢍେืूதͰ͢ʂʂʂ
   ڵຯͷ͋Δํ͸࠙਌ձʹͯ
   w ࢓ࣄɿ
   w ܾࡁ։ൃ
   w 1)1ɾ$BLF1)1ͷόʔδϣϯΞοϓ
   
3. ίʔυϨϏϡʔͰɺ
   ͜Μͳ͜ͱݴ͏ݴΘΕͨ
   ͜ͱ͋Γ·ͤΜ͔ʁ
4. ϓϧϦΫͰ͔͗͢Δ"
5. ͜ͷ࣌ɺ
   ʮͱʹ͔͘খ͍ͯ͘͜͞͠ʯ ͱ͍͏ࢦఠͰऴΘΓ͕ͪ͡Ό ͳ͍Ͱ͔͢ʁ
6. ͳͷͰ΍ΓํΛ੔ཧͯ͠Έ ·ͨ͠#
7. ໨࣍ w େ͖͍ϓϧϦΫͱ͸
   w େ͖͍ϓϧϦΫ͕ͳͥѱ͍ͷ͔
   w ͳͥେ͖͍ϓϧϦΫ͕ੜ·ΕΔͷ͔
   w Ͳ͏΍ͬͯϓϧϦΫΛখ͍͔ͯ͘͘͞͠
   w খ͖͘͞͠Εͳ͍ϓϧϦΫΛͲ͏͢Ε͹͍͍͔
8. େ͖͍ϓϧϦΫͬͯͲΜͳ ͷʁ
9. ྫ͑͹ 'JMFT
   $IBOHFE
   
   %J⒎
    
     ɾྫʢʣ !ϨϏϡʔ͔͔࣌ؒΔɾɾɾ
10. ྫ͑͹ ɾྫʢʣ 'JMFT
    $IBOHFE
     
    %J⒎
    
      !ਓྨʹ͸ϨϏϡʔͰ͖ͳ͍"
11. Ͳ͏͍͏ಛ௃͕͋Δͷ͔
12. େ͖͍ϓϧϦΫΛ෼ղͯ͠ΈΔ !ྫʢ1ʣΛ۠෼͚ͯ͠ΈΔͱ
13. େ͖͍ϓϧϦΫΛ෼ղͯ͠ΈΔ !͍Ζ͍Ζͳཁૉ͕ҰͭͷϓϧϦΫʹؚ·Ε͍ͯΔ ɾ৽ػೳ ɾόάमਖ਼ ɾϦϑΝΫλϦϯά …etc
14. ʮେ͖͍ʯϓϧϦΫͷͭͷ ಛ௃
15. ʮେ͖͍ʯϓϧϦΫͷͭͷಛ௃  ෳ਺ػೳؚ͕·Ε͍ͯΔ
     ͍ͭͰͷ࣮૷͕ࠞͬͯ͟Δ
     Ұػೳͷ࣮૷ίʔυ͕๲େ
16. ಛ௃̍ɿෳ਺ػೳؚ͕·Ε͍ͯΔ ୯ಠͰϦϦʔεͰ͖Δػೳ͕ෳ਺ؚ·ΕͯΔ ৽ػೳҰؾʹग़͢ҙຯ͋Δʁ
17. ಛ௃ɿ͍ͭͰͷ࣮૷͕ࠞͬͯ͟Δ ͍ͭͰͷϦϑΝΫλϦϯάɾόάमਖ਼͕ࠞͬͯ͟Δ ΍ͬͯ΋Β͑Δͷ͸͋Γ͕ͨ ͍͕ɾɾɾ
18. ಛ௃ɿҰػೳͷ࣮૷ίʔυ͕๲େ ҰػೳͰ݁ߏͳߦ਺ͷ࣮૷Λ͍ͯ͠Δ ͋ͷػೳͰ͔͍ͳɾɾɾ %JGG
    
     
19. ͳͥʮେ͖͍ʯϓϧϦΫ͸ ྑ͘ͳ͍ͷ͔
20. ͳͥʮେ͖͍ʯϓϧϦΫ͸ྑ͘ͳ͍ͷ͔  Өڹൣғ͕େ͖͘ͳΔ
     ໰୊͕ൃੜ͢ΔՄೳੑ͕ߴ·Δ
     ϨϏϡϫʔͷෛ୲͕େ͖͍
21. ཧ༝̍ɿӨڹൣғ͕େ͖͘ͳΔ No code is the best way to write secure and reliable applications. IUUQTHJUIVCDPNLFMTFZIJHIUPXFSOPDPEF ίʔυߦ਺͕ଟ͚Ε͹ଟ͍΄Ͳɺ
    αʔϏε΁ͷӨڹൣғ͸େ͖͘ͳΔ
22. ཧ༝ɿ໰୊͕ൃੜ͢ΔՄೳੑ͕ߴ·Δ Өڹൣғ͕େ͖͚Ε͹໰୊͕ൃੜ͢ΔՄೳੑ΋ߴ·Δ
23. ཧ༝ɿϨϏϡϫʔͷෛ୲͕େ͖͍ w ϨϏϡʔ͢Δྔ͕ଟ͍ͱͦΕ͚͕ͩ࣌ؒऔΒΕΔ w ʮϦϦʔεͯ͠΋໰୊ͳ͍ʯͱ൑அΛ͢Δ೴Έͦͷෛՙ Θ͔ΒΜ
24. ͳͥେ͖͘ͳΔͷ͔
25. Ұؾʹ΍Ζ͏ͱ͢Δ͔Β"
26. ͡Ό͋Ͳ͏͢Ε͹͍͍ͷ͔
27. খ͍͞ཁૉʹ੾ͬͯࡉ͔ ͘ϦϦʔε͢Δ#
28. খ͍͞ཁૉʹ੾ͬͯࡉ͔͘ϦϦʔε͢Δ #FGPSF
    શཁૉΛҰؾʹϨϏϡʔɾϦϦʔε "GUFS
    খ͍͞ཁૉʹ੾ͬͯࡉ͔͘ϨϏϡʔɾϦϦʔε
29. Ͳ͏΍ͬͯ੾Γ෼͚͍ͯ͘ ͔⚔
30. Ͳ͏΍ͬͯখ͘͞੾Γ෼͚͍͔ͯ͘ w 45&1
    είʔϓΛఆٛ͢Δ
    w 45&1
    είʔϓ֎Λ੾Γग़͢
    w 45&1
    είʔϓ಺Λࡉ෼Խ͢Δ
    w 45&1
    είʔϓ಺͔Β੾Γग़͢
31. 45&1
    είʔϓΛఆٛ͢Δ ʮͲ͜·ͰΛ΍Βͳ͚Ε͹͍͚ͳ͍ͷ͔ʯ
    .VTUͳ෦෼Λఆٛ͢Δ #FGPSF "GUFS
32. 45&1
    είʔϓ֎͸੾Γग़͢ είʔϓ֎͸ϓϧϦΫ͔Β੾Γग़͢
    ϦϦʔεՄೳ୯Ґʹͯ͠ࣄલϦϦʔε
33. 45&1
    είʔϓ಺Λࡉ෼Խ͢Δ ࣄલϦϦʔεͰ͖Δ୯Ґʹࡉ෼Խ͍ͯ͘͠ 
    طଘͷৼΔ෣͍Λม͑ͳ͍ൣғ
34. 45&1
    είʔϓ಺Λࡉ෼Խ͢Δ ͦΕͧΕͷػೳ͝ͱʹෳ਺ͷΫϥεʹରͯ͠ɺ
    मਖ਼ΛՃ͍͑ͯΔέʔε͕ଟ͍
35. 45&1
    είʔϓ಺Λࡉ෼Խ͢Δ طଘͷৼΔ෣͍ʹӨڹ͕ͳ͍ൣғΛɺ
    ੾Γग़͠ର৅ͱ͢Δɻ ྫʣ
    ɾ৽نΫϥε
    ɾ*/065͕มΘΒͳ͍طଘϝιουͷ मਖ਼
    ʜFUD
36. 45&1
    είʔϓ಺Λࡉ෼Խ͢Δ
    ྫʣ৽نΫϥεɾطଘϝιουͷमਖ਼ w ৽نΫϥεͷ࡞੒
    طଘͷৼΔ෣͍΁ͷӨڹൣғ͸গ ͳ͍
    ˠ੾Γ෼͚ର৅
    ɾطଘΫϥεͷमਖ਼
    طଘͷৼΔ෣͍΁ͷӨڹൣғ͕͋ Γɻ
    ϝιουΛमਖ਼͍ͯ͠ΔΑ͏ͳ৔ ߹͸ɺطʹϝιουΛࢀর͍ͯ͠ ΔՕॴͷಈ࡞อূ͕ඞཁɻ
37. 45&1
    είʔϓ಺͔Β੾Γग़͢
    #FGPSF "GUFS
38. 45&1ऴྃޙ #FGPSF "GUFS
39. ͱ͸͍͑
40. ͦΕͰ΋Ͱ͔͍΋ͷ͸Ͱ͔ ͍
41. খ͖͘͞͠Εͳ͍ϓϧϦΫ ͸Ͳ͏͢Ε͹͍͍͔
42. ࡉ͔͍୯ҐͰͷϨϏϡʔ#
43. ࡉ͔͍୯ҐͰͷϨϏϡʔ #FGPSF
    ·ͱΊͯ࠷ޙʹϨϏϡʔɾϦϦʔε "GUFS
    ͋Δఔ౓ͷ·ͱ·Γ͝ͱʹϨϏϡʔ
44. ͋Δఔ౓ͷ·ͱ·Γ͝ͱʹϦϦʔε w ΍Γํ
    w τϐοΫϒϥϯνʢҊ݅ઐ༻ϒϥϯνʣΛ׆༻͢Δ
    w ౎౓ɺ೿ੜϒϥϯνΛ੾ͬͯ։ൃ͢Δ
    w خ͍͜͠ͱ
    w ϨϏϡʔ୯Ґ͕Θ͔Γ΍͘͢ͳΔ
    w ʢHJUIVCʣϨϏϡϫʔͷίϝϯτΛ໯͍΍͍͢
    w Ͳ͜·ͰϨϏϡʔࡁΈ͔͕Ұ໨ྎવ
    
45. ·ͱΊ w ෳ਺ͷཁૉΛҰؾʹ΍Δ͔ΒϓϧϦΫ͸େ͖͘ͳΔ
    w খ͘͞੾ͬͯࡉ͔͘ϦϦʔε͍ͯ͘͠
    w খ͘͞੾Γ෼͚Δʹ͸ͭͷ45&1Λ౿Ή
    w 45&1
    είʔϓΛఆٛ͢Δ
    w 45&1
    είʔϓ֎Λ੾Γग़͢
    w 45&1
    είʔϓ಺Λࡉ෼Խ͢Δ
    w 45&1
    είʔϓ಺͔Β੾Γग़͢
    w খ͖͘͞͠Εͳ͍৔߹͸͜·ΊͳϨϏϡʔΛ͢Δ
46. ͦͷଞৄ͘͠͸ϒϩάʹͯ# ϨϏϡʔΛ΋Β͍΍͍͢ࡉ͔͍ϓϧϦΫͷ੾Γ෼͚ํ
47. #"4&גࣜձࣾ
    
    ౦ޱ࿨ᏻʢ!,IJHBTIJHVDIJʣ
    ͝ਗ਼ௌ͋Γ͕ͱ͏͍͟͝·ͨ͠ʂ