Review all the things!

The world has gone crazy. It has made us extremely busy nowadays. Product owners want us to write code fast, to ship new features even faster. Because of that we don't have time for self-improvement.

We are programmers that may be great in their craft, that may write top notch source code, but we are also - humans. We make mistakes, all the time. We also want to become better people, better engineers. But how can we do that? Why should we review our work?

You know, I have a vision, where unit testing and code review are a standard, not an opt-in feature in iOS development. I've been fighting a lot to introduce them at my workplace. Let me tell you what things get reviewed in other industries and why it is so. Join the talk if you want to start code review practices at your company, but you don't know how to kick it off.

Recording: https://www.youtube.com/watch?v=KRVdrwxIszU

Transcript

1. Review all the things! Maciej Piotrowski
2. swifting.io
3. swifting.io
4. importance of review
5. I’ve done 100s of reviews
6. it’s important
7. I’ve learnt from an expert
8. real-life example
12. review code
14. 2 important aspects
15. ego effect
16. good culture
17. how to introduce it at work?
18. unit testing code review
19. not taught
22. coding task
23. pull-what?
24. pull-request
25. git flow
33. unit testing + code review = ♥
34. obstacles
35. 3 months, 3 developers usual estimate
36. 30 min, not 3 days for a bug search for a review
37. fire and
    forget projects
38. review and
    learn projects
39. there is no time
40. there is no try just do it do or do not
41. but it takes too much time
42. how much time to deliver a drug?
43. how much time to deliver a drug? 1 year ?
44. how much time to deliver a drug? 3 years ?
45. on average 10 years
46. Is 30 min for a code review too long?
47. clinical trial
48. software validation
50. peer reviewed academic journals
52. not everyone do it
53. should we review?
57. why should we?
58. review and
    learn improve teach
59. good habits stay
61. Review all the things!
62. swifting.io
63. • code review @ swifting.io • Confusion String Quartet • The Drug Approval Process - Medscape • Mobile Academy • NASA images • Pixabay