Slide 1

Slide 1 text

Code Review It’s not about you…

Slide 2

Slide 2 text

Kick off 1. Getting a degree in intellectual property taught me that: a. All artists are prostitutes 2. Also, remember that “Code is art”

Slide 3

Slide 3 text

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<<*

Slide 4

Slide 4 text

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

Slide 5

Slide 5 text

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

Slide 6

Slide 6 text

You don’t have time To watch YoutTube?

Slide 7

Slide 7 text

If it pointless - you made it so! (And I will choke you if you keep talking like that) We do not argue - we discuss

Slide 8

Slide 8 text

You are taking responsibility anyway <- Frank Herbert Dune

Slide 9

Slide 9 text

Change your approach, feel safe