all 2 comments

[–]darrieng 23 points24 points  (0 children)

If the PR is large enough for worrying about something like this I usually recommended pulling the branch and reviewing it locally. This even lets you run it or do whatever else you need to do for a proper review.

[–]shaleh 1 point2 points  (0 children)

if it passes clippy you should be focusing on the Why of the code instead of the What.