How to review someone else's code

[@benzflynn]

*Reasons For Code Reviews*
1. To ensure consistent application of best practice in coding throughout the group/organization.
2. To explain each team member's code to all others: team members get better vision of entire system.
3. To avoid over-dependence on group leader for system overview or integration bugs.
4. To provide basis for discussions on code between team members so as to stimulate new ideas.
5. To avoid coder's blind spot, i.e. inability of authors to see their own coding limitations or biases.
6. To reach a more objective idea of code readability. What's clear to one coder may not be to others.
7. To amend inadequate tests for the code.

[@ChrisAthanas]

6:26 bike shedding is not nitpicking
It’s making suggestions that are “At least something” but not actually helping improve the core solution
Basically instead of addressing an actual problem, make suggestions that take the project in a different direction
Suggesting technical changes that do not address the actual problem
Seeing all problems as nails when you are an expert at hammering kind of thing

[@danilocecilia7831]

After my third video looking for a concise explanation on best practices for code reviewing , I can say you got to the point right away and really well explained, thanks for this!

[@akovac35]

Yes, the communication part is very important (2 and 5)!

[@lindasmith3978]

thank you for this! I am a Codecademy FSE student, currently at 20%

[@rockyvolcano122]

7% :0

[@EpsilonEridani_]

@@rockyvolcano122 13%! :)

[@mahmoudtaher5588]

I think that you meant `Increase Bus Factor` instead of `reduce buss factor` !! Am I missing something ?

[@nargeshaeri7028]

Thank you for sharing such nice tips on code review.

[@PierreLaBaguette]

Very informative and clear video, thank you very much for this!

[@vickyanand6483]

Just wanted to have an insight . What ate the most common problems/mistakes are pointed out by reviewers I have just got to check null pointer exception

[@hertechera]

Thank you

[@b4rt1j1]

The level of sus when boss requests code review from junior 🧐

[@TyrellJoanna]

Thank you!

[@hisfluffinessscaredy-catth9862]

Thanks!

[@JacintaDejohn]

important thnks

[@sjsamphex]

Please add CC for deaf viewers

[@armkiller2011]

thanks

[@StrangeIndeed]

thank U

[@TheBongoZeptobrewery]

Philly

[@tamilmaranc]

great

[@imjeffreylee]

this code sucks lol

[@notmemes]

ur mum sucks

[@Jonathan-hl5il]

I'm trying to learn HTML and CSS.

[@PierreLaBaguette]

How's it going?

[@Jonathan-hl5il]

@@PierreLaBaguette I'm doing okay.

[@lanyado98]

Thank you!