3 ms·
I would recommend discussing with your team. Personally, I look at what is in the review and understand why the changes are being made. Then I look at each fil
by Graffur 6y ago
I would recommend discussing with your team.
Personally, I look at what is in the review and understand why the changes are being made. Then I look at each file that is changed. I look at the code changes from a few perspectives including complexity, security, best practice and conformity to the code base. I review any test code and any static code analysis tool output.