Who, what, when, where, and why should you check out?

Question

• Who needs to be reviewed?
• Who should review?
• Which code should be reviewed? (all code? Big changes? Etc)
• Where should the review be conducted? (Should this be the case in man?)
• When should reviews be conducted? (Gradually? Before registering?)
• Why should the code be viewed?

I have some opinions on this, but I will send them an answer.

Answer 1

Who needs to be reviewed?

Everyone who sends the code to a shared repository.

Who should review?

• A mentor or senior engineer who will look for unpleasant odors and errors in the architectural correctness of the code.

• pod, , 3d- , .

• , .

? ( ? ? Etc)

, ( ).

? ( ?)

, . , ​​, CodeReviewer, , , .

CodeBear, , , , , , , , , , , 15 , , .

? (? ?)

. , , "" .: (

?

.

. , , , , , , , . 2 , 20 .

. - , . , , " " , , , . , , .

Answer 2

?

, ,

?

? ( ? ? Etc)

. . ini , .

? ( ?)

, . , , IDE, . , - , .

? (? ?)

, . , .

?

ginormous. , , . , , , -. .

Answer 3

, ... -! .

, , , , , . , , UML , QA ..

?

• .
• , .
• , .
• .
• .
• , .

Answer 4

. , , (-) , , . , , . ( .)

, .