|
|
|
|
|
by Arainach
1316 days ago
|
|
You don't remove their ability to do that. If you need more context, then there are UI elements to show the other lines. At Google, there are also links to the file in question in code search if you want to look at history or any other related context. Most changes don't need this. A prerequisite of fast code reviews is small changes. Rather than 3000-line features, make a series of changes with 10-100 lines of code plus tests. Reviews can quickly understand the change in logic and confirm that test cases for the new codepaths are being added. Wham, bam, done in two minutes. Sure, some reviews take more time, but of them 10-30 code reviews I do a week, it's perhaps 10% of them. |
|