Slide 1

Slide 1 text

Review all the things! Maciej Piotrowski

Slide 2

Slide 2 text

你好! 我叫 Maciej. 我是从波兰.

Slide 3

Slide 3 text

Halo! Nama saya Maciej. Saya berasal dari Poland.

Slide 4

Slide 4 text

Hi! I’m Maciej. I’m from Poland.

Slide 5

Slide 5 text

next time in Tamil … ;)

Slide 6

Slide 6 text

No content

Slide 7

Slide 7 text

swifting.io

Slide 8

Slide 8 text

importance of review

Slide 9

Slide 9 text

I’ve done 100s of reviews

Slide 10

Slide 10 text

it’s important

Slide 11

Slide 11 text

real-life example

Slide 12

Slide 12 text

No content

Slide 13

Slide 13 text

review music

Slide 14

Slide 14 text

review code

Slide 15

Slide 15 text

No content

Slide 16

Slide 16 text

3 types pair programming formal inspection show me

Slide 17

Slide 17 text

formal inspection

Slide 18

Slide 18 text

pair programming

Slide 19

Slide 19 text

show me your code

Slide 20

Slide 20 text

3 important aspects checklist ego effect positive culture

Slide 21

Slide 21 text

checklist

Slide 22

Slide 22 text

ego effect

Slide 23

Slide 23 text

positive culture

Slide 24

Slide 24 text

how to introduce it at work?

Slide 25

Slide 25 text

unit testing code review

Slide 26

Slide 26 text

No content

Slide 27

Slide 27 text

No content

Slide 28

Slide 28 text

coding task

Slide 29

Slide 29 text

pull-what?

Slide 30

Slide 30 text

pull-request

Slide 31

Slide 31 text

git flow

Slide 32

Slide 32 text

No content

Slide 33

Slide 33 text

No content

Slide 34

Slide 34 text

No content

Slide 35

Slide 35 text

No content

Slide 36

Slide 36 text

No content

Slide 37

Slide 37 text

No content

Slide 38

Slide 38 text

No content

Slide 39

Slide 39 text

unit testing + code review = ♥

Slide 40

Slide 40 text

obstacles

Slide 41

Slide 41 text

3 months, 3 developers usual estimate

Slide 42

Slide 42 text

1 hour, to avoid 1 week for a bug search for a review

Slide 43

Slide 43 text

fire andforget projects

Slide 44

Slide 44 text

review andlearn projects

Slide 45

Slide 45 text

there is no time

Slide 46

Slide 46 text

there is no try just do it do or do not

Slide 47

Slide 47 text

but it takes too much time

Slide 48

Slide 48 text

Is 60 min for a code review too much?

Slide 49

Slide 49 text

how much time to deliver a drug?

Slide 50

Slide 50 text

how much time to deliver a drug? 1 year ?

Slide 51

Slide 51 text

how much time to deliver a drug? 3 years ?

Slide 52

Slide 52 text

on average 10 years

Slide 53

Slide 53 text

clinical trial

Slide 54

Slide 54 text

software validation

Slide 55

Slide 55 text

No content

Slide 56

Slide 56 text

peer reviewed academic journals

Slide 57

Slide 57 text

No content

Slide 58

Slide 58 text

not everyone does it

Slide 59

Slide 59 text

should we review?

Slide 60

Slide 60 text

No content

Slide 61

Slide 61 text

No content

Slide 62

Slide 62 text

No content

Slide 63

Slide 63 text

why should we?

Slide 64

Slide 64 text

review andlearn improve teach

Slide 65

Slide 65 text

good habits stay

Slide 66

Slide 66 text

No content

Slide 67

Slide 67 text

Review all the things!

Slide 68

Slide 68 text

swifting.io

Slide 69

Slide 69 text

• code review @ swifting.io • The Drug Approval Process - Medscape • Mobile Academy • NASA images • Pixabay