It adds so much extra and he’ll push thousands of lines of code into a PR every week. He had one bug and tried to refactor it, he bloated that single file by 16%.
It adds so much extra and he’ll push thousands of lines of code into a PR every week.
This is just a waste of the reviewer’s own time that could be better spent doing actual work. Is it at least split up into multiple commits, or is it one giant shitshow?
I have an engineer that uses it heavily.
It adds so much extra and he’ll push thousands of lines of code into a PR every week. He had one bug and tried to refactor it, he bloated that single file by 16%.
It’s almost impossible to review.
This is just a waste of the reviewer’s own time that could be better spent doing actual work. Is it at least split up into multiple commits, or is it one giant shitshow?