|
|
|
|
|
by binsquare
859 days ago
|
|
This is one of those cases of where we're focusing on the wrong metric for a very generic problem. There's too many facets to what makes a PR good that boiling it down to line count is silly. Ex. Small code changes are required if the solution requires it - forcing 5 lines change to 50 lines would be considered wildly bad idea. On the other hand a 500 line PR to solve one problem suddenly broken down to 10 PRs makes the review far harder depending on the context. |
|