Slide 1

Slide 1 text

RESCUE SQUAD RAILS EDITION Ancient City Ruby 2015

Slide 2

Slide 2 text

I ❤ 'ancient' city

Slide 3

Slide 3 text

ben.lovell.720p.S01_E02.mkv

Slide 4

Slide 4 text

No content

Slide 5

Slide 5 text

No content

Slide 6

Slide 6 text

No content

Slide 7

Slide 7 text

london

Slide 8

Slide 8 text

england

Slide 9

Slide 9 text

london

Slide 10

Slide 10 text

not yoga, drunk

Slide 11

Slide 11 text

No content

Slide 12

Slide 12 text

psa: don't stare into the eclipse

Slide 13

Slide 13 text

No content

Slide 14

Slide 14 text

No content

Slide 15

Slide 15 text

No content

Slide 16

Slide 16 text

No content

Slide 17

Slide 17 text

No content

Slide 18

Slide 18 text

ben lovell ltd ™ © ® giving no fucks since 2014

Slide 19

Slide 19 text

Slide 20

Slide 20 text

rails sucks

Slide 21

Slide 21 text

$ rails new life_is_good

Slide 22

Slide 22 text

No content

Slide 23

Slide 23 text

No content

Slide 24

Slide 24 text

No content

Slide 25

Slide 25 text

No content

Slide 26

Slide 26 text

No content

Slide 27

Slide 27 text

No content

Slide 28

Slide 28 text

No content

Slide 29

Slide 29 text

No content

Slide 30

Slide 30 text

@benlovell twitter/github/irc

Slide 31

Slide 31 text

I really suck!

Slide 32

Slide 32 text

Here’s some commits I made just the other day

Slide 33

Slide 33 text

No content

Slide 34

Slide 34 text

No content

Slide 35

Slide 35 text

! "

Slide 36

Slide 36 text

Programming is like skipping merrily between car crashes

Slide 37

Slide 37 text

WHY!!!!? !

Slide 38

Slide 38 text

Ephemeral Teams

Slide 39

Slide 39 text

Make it work Make it right Make it fast

Slide 40

Slide 40 text

Make it work Make it right Make it fast

Slide 41

Slide 41 text

Is it done yet? Is it done yet? Is it done yet?

Slide 42

Slide 42 text

!!!

Slide 43

Slide 43 text

Rails Derailed

Slide 44

Slide 44 text

No content

Slide 45

Slide 45 text

No content

Slide 46

Slide 46 text

No content

Slide 47

Slide 47 text

No content

Slide 48

Slide 48 text

three hard problems in computer science

Slide 49

Slide 49 text

1. cache invalidation

Slide 50

Slide 50 text

2. naming things

Slide 51

Slide 51 text

DHH

Slide 52

Slide 52 text

No content

Slide 53

Slide 53 text

Changing requirements

Slide 54

Slide 54 text

sorry Your project is not a special ❄

Slide 55

Slide 55 text

guess what? software changes

Slide 56

Slide 56 text

Quit whining!

Slide 57

Slide 57 text

No. 1 best job?

Slide 58

Slide 58 text

No content

Slide 59

Slide 59 text

$ git clone ...

Slide 60

Slide 60 text

Huge pile of shit?

Slide 61

Slide 61 text

No content

Slide 62

Slide 62 text

No content

Slide 63

Slide 63 text

!" GTFO?

Slide 64

Slide 64 text

5 Steps

Slide 65

Slide 65 text

1 Step

Slide 66

Slide 66 text

Perception change it!

Slide 67

Slide 67 text

I see dead people

Slide 68

Slide 68 text

Being a positive influence

Slide 69

Slide 69 text

Opinions as pull requests

Slide 70

Slide 70 text

Broken windows

Slide 71

Slide 71 text

Always leave the camp cleaner than you found it

Slide 72

Slide 72 text

It’s better to seek forgiveness than ask permission

Slide 73

Slide 73 text

2 step

Slide 74

Slide 74 text

Metrics

Slide 75

Slide 75 text

CI

Slide 76

Slide 76 text

No content

Slide 77

Slide 77 text

!

Slide 78

Slide 78 text

Effect a consistent style

Slide 79

Slide 79 text

Make it approachable

Slide 80

Slide 80 text

Rubocop

Slide 81

Slide 81 text

2nd most stylish rubyist

Slide 82

Slide 82 text

$ 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

Slide 83

Slide 83 text

Auto correction?

Slide 84

Slide 84 text

~/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

Slide 85

Slide 85 text

Sorcery!

Slide 86

Slide 86 text

No content

Slide 87

Slide 87 text

Test Coverage

Slide 88

Slide 88 text

$ gem install SimpleCov

Slide 89

Slide 89 text

No content

Slide 90

Slide 90 text

You're sitting on a throne of lies

Slide 91

Slide 91 text

Cyclomatic Complexity

Slide 92

Slide 92 text

$ gem install metric_fu-Saikuro

Slide 93

Slide 93 text

Coverage + Cyclomatic Complexity Inform your tests

Slide 94

Slide 94 text

Duplication Duplication Duplication Duplication

Slide 95

Slide 95 text

$ 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

Slide 96

Slide 96 text

Not all duplication

Slide 97

Slide 97 text

Good DRY Bad DRY

Slide 98

Slide 98 text

single unambiguous authoritative representation

Slide 99

Slide 99 text

Churn

Slide 100

Slide 100 text

$ 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 | +----------------------------------------------------------+---------------+

Slide 101

Slide 101 text

All the metrics! $ gem install metric_fu cane, churn, flog, flay, reek roodi, saikuro, code statistics rails best practices

Slide 102

Slide 102 text

code climate?

Slide 103

Slide 103 text

Metric Overflow!

Slide 104

Slide 104 text

Only capture that you plan to action

Slide 105

Slide 105 text

Make it known

Slide 106

Slide 106 text

Dashboard

Slide 107

Slide 107 text

Gamify

Slide 108

Slide 108 text

3 step

Slide 109

Slide 109 text

Tests

Slide 110

Slide 110 text

% 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

Slide 111

Slide 111 text

require 'test_helper' class UserTest < ActiveSupport::TestCase # test "the truth" do # assert true # end end

Slide 112

Slide 112 text

No content

Slide 113

Slide 113 text

All kinds of bad

Slide 114

Slide 114 text

No tests Slow tests Low quality tests

Slide 115

Slide 115 text

Tests are not sacred artifacts

Slide 116

Slide 116 text

Tests can be transient

Slide 117

Slide 117 text

zero coverage? this might not be so bad

Slide 118

Slide 118 text

Feature specs go full stack or go home

Slide 119

Slide 119 text

Poor or slow units? wrong time, wrong place

Slide 120

Slide 120 text

Repeat after me: Tests are not sacred artifacts

Slide 121

Slide 121 text

kill them with fire

Slide 122

Slide 122 text

4 step

Slide 123

Slide 123 text

Seek and destroy

Slide 124

Slide 124 text

Reduce the surface area

Slide 125

Slide 125 text

Always be merciless

Slide 126

Slide 126 text

Metrics

Slide 127

Slide 127 text

Low hanging fruit Targets or goals which are easily achievable but do not require lots of effort

Slide 128

Slide 128 text

Improve Incrementally

Slide 129

Slide 129 text

No content

Slide 130

Slide 130 text

versioning rails stay on target, stay on target

Slide 131

Slide 131 text

5 step

Slide 132

Slide 132 text

Keep it full unicorn

Slide 133

Slide 133 text

Fail the build

Slide 134

Slide 134 text

Always be improving

Slide 135

Slide 135 text

Perception Metrics Tests Reduce & Improve Stay Full Unicorn

Slide 136

Slide 136 text

No content

Slide 137

Slide 137 text

No content

Slide 138

Slide 138 text

No content

Slide 139

Slide 139 text

No content

Slide 140

Slide 140 text

meet hot single sharks in your area

Slide 141

Slide 141 text

No content

Slide 142

Slide 142 text

No content

Slide 143

Slide 143 text

No content

Slide 144

Slide 144 text

No content

Slide 145

Slide 145 text

Thanks! @benlovell

Slide 146

Slide 146 text

No content