コードレビューしない

Transcript

1. ίʔυϨϏϡʔ
   ͠ͳ͍ Seiei Miyagi
2. ͸ͳ͢͜ͱ □ ⾃⼰紹介 □ □ □ □
3. Θͨ͠ • ID: hanachin、旧姓: ⽐嘉 • 沖縄⽣ 沖縄育 • • 既婚・⼀児 ⽗・三⼈家族 XXXIBOBDIJO
5. ͠͝ͱ ZBTTMBCKQ
6. ͠͝ͱ
7. ͠͝ͱ
8. ͸ͳ͢͜ͱ ✔ ⾃⼰紹介 □ □ □ □
9. ͍·ίʔυϨϏϡʔͰ
   ͳʹΛ͍ͯ͠Δͷ͔
10. 動作確認しぐさ
11. ͍·ίʔυϨϏϡʔͰ
    ͳʹΛ͍ͯ͠Δͷ͔
12. かわらないもの なんかない
13. ͍·ίʔυϨϏϡʔͰ
    ͳʹΛ͍ͯ͠Δͷ͔
14. のびしろをみつける
15. ͍·ίʔυϨϏϡʔͰ
    ͳʹΛ͍ͯ͠Δͷ͔
16. どうするどうする ?
17. ͍·ίʔυϨϏϡʔͰ
    ͳʹΛ͍ͯ͠Δͷ͔
18. きえるTODO
19. ͍·ίʔυϨϏϡʔͰ
    ͳʹΛ͍ͯ͠Δͷ͔
20. 脇道にそれない
21. ͍·ίʔυϨϏϡʔͰ
    ͳʹΛ͍ͯ͠Δͷ͔
22. いいやりかたを伝える
23. ͍·ίʔυϨϏϡʔͰ
    ͳʹΛ͍ͯ͠Δͷ͔
24. 何度でも いいやりかたを伝える
25. ͍·ίʔυϨϏϡʔͰ
    ͳʹΛ͍ͯ͠Δͷ͔
26. 何度でも いいやりかたを伝え…
27. ͍·ίʔυϨϏϡʔͰ
    ͳʹΛ͍ͯ͠Δͷ͔
28. ͍·ίʔυϨϏϡʔͰ
    ͳʹΛ͍ͯ͠Δͷ͔
29. 三桁じゃないから セーフ
30. ͸ͳ͢͜ͱ ✔ ⾃⼰紹介 ✔ □ □ □
31. Ͳ͏ͯ͠ίʔυϨϏϡʔΛ
    ͍ͯ͠Δͷ͔ ⽬的 、 誤 修正 。重要 、 全員 同 知識 共有 、 全員 守 確⽴ 。⾃分 書 他 共有 「 共同所有(collective code ownership)」 可能 。 ―― 知 97 なるほど?
32. なるほど?
33. 自分が うれしかったこと
34. Ͳ͏ͯ͠ίʔυϨϏϡʔΛ
    ͍ͯ͠Δͷ͔
35. Ͳ͏ͯ͠ίʔυϨϏϡʔΛ
    ͍ͯ͠Δͷ͔
36. 自分のことは 信頼してないけど みんなのこと 信頼してる
37. みんなのコード
38. ͸ͳ͢͜ͱ ✔ ⾃⼰紹介 ✔ ✔ □ □
39. Ͳ͏΍Ε͹ίʔυϨϏϡʔΛ
    ͠ͳͯ͘Α͘ͳΔ͔ 1. ⾃分 2. 、⼤丈夫 3. ⾃分 増
40. ࣗ෼Ͱ΍Βͳ͍ɺػցʹ΍ΒͤΔ • CircleCI 2.1 Parameterized jobs • Heroku Review Apps ←これをはなすね
41. ͱ͜ΖͰ3FENJOFϓϥάΠϯ։ൃ 3FENJOF 3.3 3.4 4.0 3BJMT 4.2 4.2 5.2
42. ͱ͜ΖͰ3FENJOFϓϥάΠϯ։ൃ SVCZ3FENJOF     ✔ ✔  ✔ ✔  ✔ ✔  ✔ ✔ ✔  ✔ ✔ ✔  ✔ ✔  ✔  ✔
43. ͱ͜ΖͰ3FENJOFϓϥάΠϯ։ൃ • Microsoft SQL Server • MySQL (MariaDB) • PostgreSQL • SQLite 3 3FENJOF͕αϙʔτ͍ͯ͠Δ3%#.4
44. 多様な 動作環境
45. 3FENJOFϓϥάΠϯ։ൃͰͷίʔυϨϏϡʔ • 機能 ＜Redmine ＞ 動 ? • ＜RDBMS ＞ 動 ? • ＜Ruby ＞ 動 ?
46. やってないから わかんない!
47. テストを動かせばわかる!
48. かばれっじ １００ぱーなら だいじょうぶ！
49. ͱ͜ΖͰ3FENJOFϓϥάΠϯ։ൃ • 組 合 実⾏ • 毎⽇ ⼈ 浦島太郎状態 特 • ⼿元 動 記録 共有・再現 • ⼈ 環境 ⼿元 ( )
50. Ͳ͏΍Ε͹ίʔυϨϏϡʔΛ
    ͠ͳͯ͘Α͘ͳΔ͔ • ⾃分 動 • Redmine/ruby/RDBMS 組 合 全部試 • 動 結果 ⾒
51. $JSDMF$*
    ͷ1BSBNFUFSJ[FE
    KPCT workflows: test: jobs: - rspec: db: mysql redmine_version: ‘4.0.1’ ruby_version: ‘2.6’
52. $JSDMF$*
    ͷ1BSBNFUFSJ[FE
    KPCT jobs: rspec: parameters: ruby_version: description: version of Ruby type: string
53. $JSDMF$*
    ͷ1BSBNFUFSJ[FE
    KPCT jobs: rspec: executor: name: ruby-<<parameters.db>> ruby_version: <<parameters.ruby_version>>
54. $JSDMF$*
    ͷ1BSBNFUFSJ[FE
    KPCT executors: ruby-mysql: parameters: ruby_version: description: version of Ruby type: string docker: - image: circleci/mysql - image: circleci/ruby-<<parameters.ruby_version>>
55. $JSDMF$*
    ͷ8PSLqPX workflows: nightly: triggers: - schedule: cron: ‘0 15 * * *’ jobs: - rspec: # ここで組み合わせを列挙して行く
56. executors: ruby-mysql: docker: - image: circleci/mysql - image: circleci/ruby-<<parameterized.ruby_version>> jobs: rspec: executor: name: ruby-<<parameters.db>> ruby_version: <<parameters.ruby_version>> workflows: test: jobs: - rspec: db: mysql redmine_version: ‘4.0.1’ ruby_version: ‘2.6’
57. Ͳ͏΍Ε͹ίʔυϨϏϡʔΛ
    ͠ͳͯ͘Α͘ͳΔ͔ ✔ CircleCI 実⾏ ✔ Redmine/ruby/RDBMS 組 合 動 ✔CircleCI 結果 ⾒ ・記録 残 わたしもできるし あなたもできる
58. ίʔυϨϏϡʔ͠ͳͯ͘Α͘ͳͬͨ • 機能 ＜Redmine ＞ 動 ? • ＜RDBMS ＞ 動 ? • ＜Ruby ＞ 動 ?
59. ࣗ෼Ͱ΍Βͳ͍ɺػցʹ΍ΒͤΔ • CircleCI 2.1 Parameterized jobs • Heroku Review Apps ←これをはなすね
60. ಈ࡞֬ೝ͙͠͞ • ⾒ ⽬ 崩 、実機 確認 • ⽂⾯ 変更 、⽂⾯ 確認 • 捨 弾 、確認 • production環境 ⼤丈夫…?
61. ಈ࡞֬ೝ͙͠͞ てもとではうごいてたし じどうテストもとおった けどコードだけ見ても不安
62. ಈ࡞֬ೝ͙͠͞ • 動 • 動 ⼿間 • 外 ⾒ ⼿間 • ⼿元 動 動
63. さわってみてだいじょーぶそう なら だいじょーぶ
64. Ͳ͏΍Ε͹ίʔυϨϏϡʔΛ
    ͠ͳͯ͘Α͘ͳΔ͔ • ⼿元 動作確認 • 実際 試 • production 同 環境 動
65. )FSPLV
    3FWJFX
    "QQT
66. Ͳ͏΍Ε͹ίʔυϨϏϡʔΛ
    ͠ͳͯ͘Α͘ͳΔ͔ ✔ Heroku Review Apps push 動 ✔ view deployment ⾒ ✔ Review App production 動 わたしもできるし あなたもできる
67. ಈ࡞֬ೝΛͯ΋ͱͰ͠ͳͯ͘Α͘ͳͬͨ • ⾒ ⽬ 崩 、実機 確認 • ⽂⾯ 変更 、⽂⾯ 確認 • 捨 弾 、確認 • production環境 ⼤丈夫…? やったー
68. Ͳ͏΍Ε͹ίʔυϨϏϡʔΛ
    ͠ͳͯ͘Α͘ͳΔ͔ 1. ⾃分 、機械 ！ 2. 、⼤丈夫 3. ⾃分 増
69. ͏·͍͔͘ͳͯ͘΋ɺେৎ෉ʂͳ࢓ֻ͚ • 問題 起 • 戻 • 変 ⼤丈夫 ←これをはなすね
70. ໰୊͕ى͖ͨΒ͙͢Θ͔ΔΑ͏ʹ͢Δ
71. ໰୊͕ى͖ͨΒ͙͢Θ͔ΔΑ͏ʹ͢Δ
72. 2分で戻せた! すごい!
73. ロールバック王
74. ͏·͍͔͘ͳͯ͘΋ɺେৎ෉ʂͳ࢓ֻ͚ • 問題 起 • 戻 • 変 ⼤丈夫 ←これをはなすね
75. ໭ͤΔΑ͏ʹ͢Δ • 戻 ⽅ 考 ・共有 • 戻
76. ໭ͤΔΑ͏ʹ͢Δ
77. ໭ͤΔΑ͏ʹϚΠάϨʔγϣϯͷ
    ϩʔϧόοΫΛςετ͢Δ bin/rails db:migrate VERSION=0
78. ͏·͍͔͘ͳͯ͘΋ɺେৎ෉ʂͳ࢓ֻ͚ • 問題 起 • 戻 • 変 ⼤丈夫 ←これをはなすね
79. ม͑ΒΕΕ͹େৎ෉ 変えたくないことはテストする
80. ม͑ΒΕΕ͹େৎ෉ 変えづらくなるのは よくない
81. ม͑ΒΕΕ͹େৎ෉ 機を待つ
82. ͏·͍͔͘ͳͯ͘΋ɺେৎ෉ʂͳ࢓ֻ͚ • ⾒落 • 問題 起 気 ⼤丈夫 • 戻 ⼤丈夫 • 変 ⼤丈夫
83. Ͳ͏΍Ε͹ίʔυϨϏϡʔΛ
    ͠ͳͯ͘Α͘ͳΔ͔ 1. ⾃分 、機械 ！ 2. 、⼤丈夫！！ 3. ⾃分 増
84. ࣗ෼Λ૿΍͢ ⼈数 増 参加 増 • 同 扱 • ⼈・ ⼈ 増 • 探 ⽅ 教 機会 増
85. コードレビューするとき ドキュメントの探し方が わからない
86. ࣗ෼Λ૿΍͢
87. ࣗ෼Λ૿΍͢
88. 記事は休まず働ける
89. ドキュメントを読む能力が 育ってしまう環境 よくなさそう
90. Ͳ͏΍Ε͹ίʔυϨϏϡʔΛ
    ͠ͳͯ͘Α͘ͳΔ͔ 1. ⾃分 、機械 ！ 2. 、⼤丈夫！！ 3. ⾃分 増 ！！！
91. ͸ͳ͢͜ͱ ✔ ⾃⼰紹介 ✔ ✔ ✔ □
92. ίʔυϨϏϡʔ͠ͳ͍Λ͢Δܾҙ • ？ 考 • 機械 働 • 発表資料 書 発表資料 働
93. ͸ͳ͢͜ͱ ✔ ⾃⼰紹介 ✔ ✔ ✔ ✔
94. ίʔυϨϏϡʔ
    ͠ͳ͍ Seiei Miyagi