3 ms·
I see your point but we found out that this way the review will be done quicker rather than having discussions on the PR, since before adopting this policy the
by _shadi 8y ago
I see your point but we found out that this way the review will be done quicker rather than having discussions on the PR, since before adopting this policy the comments usually had pseudocode in them.
anyway duo to #1 and #2 what actually ends up needing a review is the only minor or medium scope changes, and most of the time the commit will be renaming a variable/method or adding a test case.
of course if you see the PR went in the wrong direction you can still reject it.