RESCUE SQUAD: Rails Edition - Ancient City Ruby 2015

Transcript

1. RESCUE SQUAD RAILS EDITION Ancient City Ruby 2015
2. I ❤ 'ancient' city
3. ben.lovell.720p.S01_E02.mkv
7. london
8. england
9. london
10. not yoga, drunk
12. psa: don't stare into the eclipse
18. ben lovell ltd ™ © ® giving no fucks since 2014
19. ❤
20. rails sucks
21. $ rails new life_is_good
30. @benlovell twitter/github/irc
31. I really suck!
32. Here’s some commits I made just the other day
35. ! "
36. Programming is like skipping merrily between car crashes
37. WHY!!!!? !
38. Ephemeral Teams
39. Make it work Make it right Make it fast
40. Make it work Make it right Make it fast
41. Is it done yet? Is it done yet? Is it done yet?
42. !!!
43. Rails Derailed
48. three hard problems in computer science
49. 1. cache invalidation
50. 2. naming things
51. DHH
53. Changing requirements
54. sorry Your project is not a special ❄
55. guess what? software changes
56. Quit whining!
57. No. 1 best job?
59. $ git clone ...
60. Huge pile of shit?
63. !" GTFO?
64. 5 Steps
65. 1 Step
66. Perception change it!
67. I see dead people
68. Being a positive influence
69. Opinions as pull requests
70. Broken windows
71. Always leave the camp cleaner than you found it
72. It’s better to seek forgiveness than ask permission
73. 2 step
74. Metrics
75. CI
77. !
78. Effect a consistent style
79. Make it approachable
80. Rubocop
81. 2nd most stylish rubyist
82. $ gem install rubocop ~/code/frontend % rubocop Inspecting 313 files .....................................................C ...................................................... ...................................................... ...................................................... ...................................................... ........................................... Offenses: app/helpers/profile_helper.rb:2:1: C: Extra empty line detected at body beginning. 313 files inspected, 1 offense detected
83. Auto correction?
84. ~/code/frontend % rubocop -a Inspecting 313 files .....................................................C Offenses: app/helpers/profile_helper.rb:2:1: C: [Corrected] Extra empty line detected at body beginning. 313 files inspected, 1 offense detected, 1 offense corrected
85. Sorcery!
87. Test Coverage
88. $ gem install SimpleCov
90. You're sitting on a throne of lies
91. Cyclomatic Complexity
92. $ gem install metric_fu-Saikuro
93. Coverage + Cyclomatic Complexity Inform your tests
94. Duplication Duplication Duplication Duplication
95. $ gem install flay ~/code/frontend % flay app/**/*.rb -f Total score (lower is better) = 256 1) Similar code found in :call (mass = 148) app/controllers/styleguide_controller.rb:63 app/controllers/styleguide_controller.rb:221 2) Similar code found in :defn (mass = 108) app/helpers/application_helper.rb:2 app/helpers/application_helper.rb:9 app/helpers/application_helper.rb:16 app/helpers/application_helper.rb:23 app/helpers/application_helper.rb:30 app/helpers/application_helper.rb:37
96. Not all duplication
97. Good DRY Bad DRY
98. single unambiguous authoritative representation
99. Churn
100. $ gem install churn ~/code/blog % churn Files +----------------------------------------------------------+---------------+ | file_path | times_changed | +----------------------------------------------------------+---------------+ | Gemfile | 20 | | app/helpers/application_helper.rb | 14 | | app/models/article.rb | 11 | | spec/models/text_filter_spec.rb | 11 | | spec/controllers/articles_controller_spec.rb | 11 | | spec/controllers/setup_controller_spec.rb | 11 | | spec/rails_helper.rb | 10 | | app/models/blog.rb | 10 | | spec/models/article_spec.rb | 10 | | spec/controllers/admin/content_controller_spec.rb | 10 | | spec/helpers/application_helper_spec.rb | 10 | +----------------------------------------------------------+---------------+
101. All the metrics! $ gem install metric_fu cane, churn, flog, flay, reek roodi, saikuro, code statistics rails best practices
102. code climate?
103. Metric Overflow!
104. Only capture that you plan to action
105. Make it known
106. Dashboard
107. Gamify
108. 3 step
109. Tests
110. % tree spec/models spec/models ├── authentication_token_spec.rb ├── challenge_spec.rb ├── entry_spec.rb ├── facebook_token_spec.rb ├── follow_spec.rb ├── followed_list_spec.rb ├── item_spec.rb ├── list_fulfilment_criterion_spec.rb ├── list_spec.rb ├── prize_spec.rb ├── rating_spec.rb └── user_spec.rb 0 directories, 12 files
111. require 'test_helper' class UserTest < ActiveSupport::TestCase # test "the truth" do # assert true # end end
113. All kinds of bad
114. No tests Slow tests Low quality tests
115. Tests are not sacred artifacts
116. Tests can be transient
117. zero coverage? this might not be so bad
118. Feature specs go full stack or go home
119. Poor or slow units? wrong time, wrong place
120. Repeat after me: Tests are not sacred artifacts
121. kill them with fire
122. 4 step
123. Seek and destroy
124. Reduce the surface area
125. Always be merciless
126. Metrics
127. Low hanging fruit Targets or goals which are easily achievable but do not require lots of effort
128. Improve Incrementally
130. versioning rails stay on target, stay on target
131. 5 step
132. Keep it full unicorn
133. Fail the build
134. Always be improving
135. Perception Metrics Tests Reduce & Improve Stay Full Unicorn
140. meet hot single sharks in your area
145. Thanks! @benlovell