|
|
|
|
|
by imetatroll
1572 days ago
|
|
If you want to approach the highest quality code your team can manage, you want: - the author to test and have a succinct PR description that includes before and after screenshots/video as appropriate - the reviewer to test the code locally The reviewer test is particularly important for front-end code which is where automated tests are more likely to be poorly written or absent. It goes without saying that there should be targeted automated tests, but I find that in practice the front-end is where this tends to fall short. |
|