Code Review - It is not about you (it is about me)

A LT about code review - why people are sometimes scared of it/don't want to do it and how to deal with it and have a more healthy approach. Heavily based on what I am saying as a speaker, with slides being just a visual help, but I do enjoy this talk anyway.

Transcript

1. Code Review It’s not about you…
2. Kick off 1. Getting a degree in intellectual property taught me that: a. All artists are prostitutes 2. Also, remember that “Code is art”
3. Do not be scared of reviewing/getting a review! I’ve seen too many people be scared of publishing Pull Requests or writing a review because: 1. I do not want people to be mean to me 2. I do not want to be mean to people 3. I’ve got not time (both to review, or get reviewed) 4. It is pointless 5. I don’t want to argue 6. I do not want to be held accountable for not my code, that I approve 7. I’m a little bi<<*
4. Your code? Is it? 1. Your code is most likely from Stack Overflow or AI, anyway so it is not yours 2. Even if it is not, it stops being just yours when it gets merged 3. You are selling it anyway you moron! 4. It is a part of a bigger codebase, shared codebase. It is everyone's responsibility
5. How no to be mean? 1. Do not treat the code you write, as “your code” 2. Do not get emotionally attached to code 3. Do not treat someones else code as just “their code” 4. Do not use personal language to receiver, make it about YOU 5. If you are suggesting a change, always explain why 6. Explain your PR’s
6. You don’t have time<? To watch YoutTube?
7. If it pointless - you made it so! (And I will choke you if you keep talking like that) We do not argue - we discuss
8. You are taking responsibility anyway <- Frank Herbert Dune
9. Change your approach, feel safe