Enter the OSS world [RuboCop] II. lost boundary

023b04c98f39cc041293d780352432ff?s=47 Koichi ITO
December 09, 2017
Programming
1
2.1k

Enter the OSS world [RuboCop] II. lost boundary

Rails Developers Meetup 2017
https://railsdm.github.io/2017/

023b04c98f39cc041293d780352432ff?s=128

Koichi ITO

December 09, 2017
Tweet

More Decks by Koichi ITO

See All by Koichi ITO
023b04c98f39cc041293d780352432ff?s=48 koic
0
1.7k
023b04c98f39cc041293d780352432ff?s=48 koic
1
1.2k
023b04c98f39cc041293d780352432ff?s=48 koic
1
280
023b04c98f39cc041293d780352432ff?s=48 koic
1
210
023b04c98f39cc041293d780352432ff?s=48 koic
1
880
023b04c98f39cc041293d780352432ff?s=48 koic
4
4.3k
023b04c98f39cc041293d780352432ff?s=48 koic
0
1k
023b04c98f39cc041293d780352432ff?s=48 koic
2
1.3k
023b04c98f39cc041293d780352432ff?s=48 koic
3
2.8k

Other Decks in Programming

See All in Programming
Bf3acef686273e03fa4fdf125fdad3e9?s=48 kuncevic
0
100
29bfc9e2bc82b775a30af70be3196386?s=48 hannaprinz
0
100
4d78b749c315e21a956053dcbf0508a9?s=48 po3rin
1
120
770ede26fb450c104128e35ac37711fd?s=48 musaprg
0
700
0fb2ce77513181750c384115cd0bc937?s=48 ksukenobe
0
110
B1412c9ed55333c1df561f64dfad69d3?s=48 pantuza
1
290
719435d98d452de7ac367c828266cf01?s=48 erikaheidi
5
3.5k
354271902cd8ba2762d05b251dfa0f84?s=48 pluu
0
260
Fa54d97af93719f9aa82873de3acb3b3?s=48 decoc
1
390
Db8efd836c9a09b71e3d8e1c60d6ea84?s=48 colinfay
0
130
0ca0c0ec6efe3a7f5220332a910e6da0?s=48 shinyoshiaki
1
490
Db2247a5b4226e1f6f74a9bd87810090?s=48 hiroakis
3
490

Featured

See All Featured
D09fad3e36ae6841f54820be0e907f7a?s=48 rocio
154
11k
D8fc2580cfaca035f666d9e4ee79a7f7?s=48 mongodb
23
3.7k
48c098b6579220a67a6ba949be6e4b5a?s=48 revolveconf
195
9.4k
95d9f37115fbb0d13135d0f77132f856?s=48 morganepeng
86
10k
170b760e0147792d0140e59ec78e9893?s=48 eitanlees
105
9.4k
64827b31aef81498662e8af4bd6d5157?s=48 jonrohan
1020
350k
Aebc2d07a50011e2a30d38435fde564a?s=48 jrom
113
6.8k
6804f1775cb4babfcc3851298566fbce?s=48 tanoku
256
24k
79acae287842acf29084005533a7fb3b?s=48 hursman
103
9k
B47b288784fda77a94aeb234ca743c24?s=48 keavy
101
14k
245cee81a9c424266e5e401d844ea881?s=48 lara
587
60k
433acaea1012b25d97ae66da9b998dad?s=48 malarkey
189
8.2k

Transcript

  1. &OUFSUIF044XPSME <3VCP$PQ> **MPTUCPVOEBSZ &OUFSUIF044XPSME <3VCP$PQ> **MPTUCPVOEBSZ &OUFSUIF044XPSME <3VCP$PQ> **MPTUCPVOEBSZ &OUFSUIF044XPSME

    <3VCP$PQ> **MPTUCPVOEBSZ &OUFSUIF044XPSME <3VCP$PQ> **MPTUCPVOEBSZ 5&$)1-":4)*#6:" ,PJDIJ*50FTN  4BU 3BJMT%FWFMPQFST.FFUVQ 5&$)1-":4)*#6:" ,PJDIJ*50FTN  4BU 3BJMT%FWFMPQFST.FFUVQ 5&$)1-":4)*#6:" ,PJDIJ*50FTN  4BU 3BJMT%FWFMPQFST.FFUVQ 5&$)1-":4)*#6:" ,PJDIJ*50FTN  4BU 3BJMT%FWFMPQFST.FFUVQ 5&$)1-":4)*#6:" ,PJDIJ*50FTN  4BU 3BJMT%FWFMPQFST.FFUVQ
  2. None
  3. None

  4. 8FMDPNFGFFECBDL (FNpMF HJU@TPVSDF HJUIVC \cSFQPcIUUQTHJUIVCDPN\SFQP^HJU^ ! HFNSBJMT HJUIVCSBJMTSBJMT HFNBDUJWFSFDPSEPSBDMF@FOIBODFEBEBQUFS HJUIVCbSTJNPSBDMF

    FOIBODFE Preparing for upgrade to Rails 5.2 uses master branch instead of beta.
  5. None
  6. None

  7. 0DUPCFS  3VCP$PQSFMFBTFE

  8. None

  9. ೔ࠒ࢖͍ͬͯΔ044ͰͷൃݟΛ VQTUSFBNʹؐݩ͢Δ࿩ &QJTPEFT 1BSU* 1BSU** 1BSU*** ιʔγϟϧίʔσΟϯάΛ௨ͯ͡Ͳ͏ Ϣʔβʔ͕࢖͏ܗʹͳ͍͔ͬͯ͘ͷ࿩

  10. &QJTPEFT 1BSU* ೔ࠒ࢖͍ͬͯΔ044ͰͷൃݟΛ VQTUSFBNʹؐݩ͢Δ࿩ 1BSU** 1BSU*** ιʔγϟϧίʔσΟϯάΛ௨ͯ͡Ͳ͏ Ϣʔβʔ͕࢖͏ܗʹͳ͍͔ͬͯ͘ͷ࿩

  11. Enter the OSS world RuboCop I. presage comment

  12. 1SFNPOJUJPO

  13. "3BJMTBQQMJDBUJPO EFWFMPQNFOU

  14. 3FWJFXDPNNFOU

  15. ࣗ෼ͨͪͷࢦఠͨ͜͠ͱ͸ ଞͷػձͰ΋ࢦఠ͞Ε͏Δ

  16. None

  17. 044 ৘ใԽٕज़Λ௨ͯ͡ʜ 3JEFPO044

  18. IUUQTUXJUUFSDPNPOLTUBUVT

  19. &QJTPEFT 1BSU* ೔ࠒ࢖͍ͬͯΔ044ͰͷൃݟΛ VQTUSFBNʹؐݩ͢Δ࿩ 1BSU** 1BSU*** ιʔγϟϧίʔσΟϯάΛ௨ͯ͡Ͳ͏ Ϣʔβʔ͕࢖͏ܗʹͳ͍͔ͬͯ͘ͷ࿩

  20. ೔ࠒ࢖͍ͬͯΔ044ͰͷൃݟΛ VQTUSFBNʹؐݩ͢Δ࿩ &QJTPEFT 1BSU* 1BSU** ιʔγϟϧίʔσΟϯάΛ௨ͯ͡Ͳ͏ Ϣʔβʔ͕࢖͏ܗʹͳ͍͔ͬͯ͘ͷ࿩ 1BSU***

  21. Enter the OSS world RuboCop II. lost boundary

  22. 044ʹ΋໰୊͸ͦΕͳΓʹ͋Γɺ ΞϓϦέʔγϣϯ͔Βܨ͕͍ͬͯ ΔҎ্ɺଞਓࣄͷίʔυͰ͸ͳ͍ 3JEFPO044

  23. ؔΘΓํͷελΠϧྫ w۩ମతʹࠔ͍ͬͯΔ໰୊ w͓΋͠Ζͦ͏ wແཧ͠ͳ͍ όʔϯΞ΢τ͠ͳ͍Α͏τϦΞʔδ͢Δ

  24. $MJDLlCVHzMBCFM

  25. 3BJMT).PS)0%FQFOEFOU

  26. 8IBUJTUIJT$PQ

  27. CCBUTPWSVCPDPQ <'JY>'JYA3BJMT)BT.BOZ0S)BT0OF%FQFOEFOUA JODPSSFDUMZqBHTAXJUI@PQUJPOTACMPDLT

  28. 8SJUFSFQSPEVDUJPOTQFDT DPOUFYUXJUI@PQUJPOTEFQFOEFOUEFTUSPZEP JUEPFTOPUSFHJTUFSBOP⒎FOTFEP FYQFDU@OP@P⒎FOTFT 36#:TUSJQ@JOEFOU  DMBTT1FSTPO XJUI@PQUJPOTEFQFOEFOUEFTUSPZEP IBT@POFGPP FOE

    FOE 36#: FOE FOE # Goal: False positive don't occur below ! class Person with_options dependent: :destroy do has_one :foo end end

  29. XJUI@PQUJPOTPG"45 SVCZQBSTFFXJUI@PQUJPOTEFQFOEFOUEFTUSPZEPcBTTPDc BTTPDIBT@POFGPPFOE` ! CMPDL  TFOEOJMXJUI@PQUJPOT  IBTI 

    QBJS  TZNEFQFOEFOU   TZNEFTUSPZ   BSHT  QSPDBSHBTTPD   TFOE  MWBSBTTPD IBT@POF  TZNGPP

  30. XJUI@PQUJPOTPG"45 QPXFSFECZIUUQTHJUIVCDPNLPJDSVCZ@BTU@WJTVBMJ[FS

  31. *NQMFNFOUBUJPO 3FE (SFFO 3FGBDUPS 5FTUDPEFJTNPSFPCWJPVTBOEFBTJFSUP VOEFSTUBOEUIBOUIFPQFSBUJPOPG"45

  32. 0QFOB13

  33. $POWFSTBUJPO AJT@IBT@NBOZ@PS@IBT@POF@XJUIPVU@PQUJPOT AJT BOFYJTUJOHNFUIPEOBNF

  34. "CPVU/BNJOH1SFEJDBUF/BNF

  35. EFG@OPEF@NBUDIFS EFG@OPEF@NBUDIFSJT@YYY 1"55&3/  TFOEOJM \IBT@NBOZIBT@POF^@  1"55&3/ def_node_matcher dynamic

    defines a method which applies a pattern to an AST node. /PUF/BNJOH1SFEJDBUF/BNFDPQEPFTOPUBXBSFPG EZOBNJDBMMZEFpOFENFUIPEPSMPXFS -JLFEFpOF@NFUIPE EFpOF@TJOHMFHPO@NFUIPE

  36. $POWFSTBUJPO

  37. "OUJGBU13

  38. ผ໰୊ͱͯ͠੾Γग़ͤΔ Α͏Ͱ͋Ε͹ผͷ13Ͱ ରॲ͢ΔΑ͏ʹ͍ͯ͠Δ ΋ͱ΋ͱજΜͰ͍ͨ໰୊ͱͷରቂ ΋ͪΖΜͦͷ13Ͱͷมߋ͕ద੾Ͱ͋Ε͹ɺͦͷ13Ͱ ରԠ͢Δ

  39. CCBUTPWSVCPDPQ "EEOFXA*OUFSOBM"⒎BJST%ZOBNJD1SFEJDBUF/BNFADPQ

  40. 5IFpSTUEFTJHO /BNJOH 1SFEJDBUF/BNF &YJTUJOH DPQDMBTT

  41. 5IFpSTUEFTJHO /BNJOH 1SFEJDBUF/BNF NPEVMF $POpHVSBCMF 1SFEJDBUF/BNJOH /BNJOH 1SFEJDBUF/BNF NJYJO &YUSBDU

    NPEVMF $IFDLTUBUJD NFUIPET &YJTUJOH DPQDMBTT 4IBSF DPOpHVSBUJPO

  42. 5IFpSTUEFTJHO /BNJOH 1SFEJDBUF/BNF NPEVMF $POpHVSBCMF 1SFEJDBUF/BNJOH /BNJOH 1SFEJDBUF/BNF NJYJO *OUFSOBM"⒎BJST

    %ZOBNJD 1SFEJDBUF/BNF NJYJO &YUSBDU NPEVMF "QQFOE OFXDMBTT $IFDLTUBUJD NFUIPET $IFDLEZOBNJD NFUIPET &YJTUJOH DPQDMBTT 4IBSF DPOpHVSBUJPO

  43. *NQMFNFOUBUJPOUFSN QPXFSFECZIUUQTHJUIVCDPNZZBHJKQDBM

  44. 0QFOB13

  45. None

  46. $POWFSTBUJPO

  47. $POWFSTBUJPO

  48. *OUFSOBM"⒎BJSTEFQBSUNFOU JOUFSOBM"1*  ˣ /BNJOHEFQBSUNFOU QVCMJTIFE"1* .BLFTFOTF✨

  49. CCBUTPWSVCPDPQ "EEA.FUIPE%FpOF.BDSPTAPQUJPOUP A/BNJOH1SFEJDBUF/BNFADPQ

  50. 0QFOB13

  51. 5IFTFDPOEEFTJHO /BNJOH 1SFEJDBUF/BNF /BNJOH 1SFEJDBUF/BNF "EE .FUIPE%FpOF.BDSPT PQUJPOUPDPOpHVSBUJPO *OUFSOBM *NQSPWFNFOU

    &YJTUJOH DPQDMBTT

  52. %FpOFEFGBVMUQVCMJTIFE"1* DPOpHEFGBVMUZNM /BNJOH1SFEJDBUF/BNF .FUIPEEFpOJUJPONBDSPTGPSEZOBNJDBMMZHFOFSBUFENFUIPET .FUIPE%FpOJUJPO.BDSPT EFpOF@NFUIPE EFpOF@TJOHMFUPO@NFUIPE RuboCopར༻ऀ͕࢖͏σϑΥϧτͷެ։ϝ ιουΛconfig/default.ymlʹఆٛ͢Δ

  53. %FpOFJOUFSOBM"1* SVCPDPQZNM /BNJOH1SFEJDBUF/BNF .FUIPEEFpOJUJPONBDSPTGPSEZOBNJDBMMZHFOFSBUFENFUIPE .FUIPE%FpOJUJPO.BDSPT EFpOF@NFUIPE EFpOF@TJOHMFUPO@NFUIPE EFG@OPEF@NBUDIFS EFG@OPEF@TFBSDI RuboCopࣗମͷ։ൃͰ࢖͏಺෦తͳϝιου

    ΛՃ͑ͯ.rubocop.ymlʹఆٛ͢Δ

  54. w ར༻ऀʹରͯ͠΋ಈతͳϝιο υఆٛ΁ͷνΣοΫΛҰ؏͠ ͯͰ͖ΔΑ͏ʹͳͬͨ w ಈతͳϢʔβʔϝιουఆٛ ͕૿͑ΔΑ͏Ͱ͋Ε͹ઃఆΛ ௥Ճ͢Ε͹Α͍ *NQSPWFEEFTJHO

  55. None

  56. w 044DPNNVOJUZJT $PODMVTJPO

  57. 044 1SPQSJFUBSZ 4PDJBM 6OTPDJBM FH"OBQQMJDBUJPO EFWFMPQNFOU BGUFS(JU)VC FH"OBQQMJDBUJPO EFWFMPQNFOU CFGPSF(JU)VC

    *UJTPQFOUPUIF QVCMJDVOEFS044 MJDFOTFCVUUIFSFJT OPEJBMPHVF 044Y6OTPDJBM1VCMJDSFQP 044DPNNVOJUZ

  58. 044 1SPQSJFUBSZ 4PDJBM 6OTPDJBM FH"OBQQMJDBUJPO EFWFMPQNFOU BGUFS(JU)VC FH"OBQQMJDBUJPO EFWFMPQNFOU CFGPSF(JU)VC

    *UJTPQFOUPUIF QVCMJDVOEFS044 MJDFOTFCVUUIFSFJT OPEJBMPHVF 044Y4PDJBM044DPNNVOJUZ 044DPNNVOJUZ

  59. SVCZPOSBJMTPSH "O044DPNNVOJUZ

  60. None

  61. $MJDLIFSF

  62. None
  63. None

  64. w ϓϩδΣΫτͰѻ͍ͬͯΔίʔυ ͱ044ͷίʔυ͸஍ܨ͕ΓͰ͋Δ &OUFSUIF044XPSME  w Πϯλʔωοτͱ͍͏ڥքͷͳ͍ ੈքͰ΍ΓͱΓ͢Δ͓΋͠Ζ͞ w ࠷ॳͷҰา͕೉͚͠Ε͹044(BUF

    ͳͲ΁ͷࢀՃ͔Β࢝ΊΔͷ΋Ұͭ ·ͱΊ

  65. ࣍ճ༧ࠂ

  66. ೔ࠒ࢖͍ͬͯΔ044ͰͷൃݟΛ VQTUSFBNʹؐݩ͢Δ࿩ &QJTPEFT 1BSU* 1BSU** ιʔγϟϧίʔσΟϯάΛ௨ͯ͡Ͳ͏ Ϣʔβʔ͕࢖͏ܗʹͳ͍͔ͬͯ͘ͷ࿩ 1BSU***

  67. ೔ࠒ࢖͍ͬͯΔ044ͰͷൃݟΛ VQTUSFBNʹؐݩ͢Δ࿩ &QJTPEFT 1BSU* 1BSU** ιʔγϟϧίʔσΟϯάΛ௨ͯ͡Ͳ͏ Ϣʔβʔ͕࢖͏ܗʹͳ͍͔ͬͯ͘ͷ࿩ 1BSU***

  68. &OUFSUIF044XPSME <3VCP$PQ> *** &OUFSUIF044XPSME <3VCP$PQ> *** &OUFSUIF044XPSME <3VCP$PQ> ***

  69. l Ұ೥ͷ͏ͪ೔ίʔυॻ͖ɺ 3VCZ,BJHJͳͲͷ࢒Γ೔ؒ Ͱ։ൃऀͨͪͱͦͷ࿩Λ͢Δ ʔ3VCZ,BJHJ1BSUZΑΓ

  70. FOE