| ▲ | mullingitover 4 days ago | |||||||
> expects the “code review” process to handle the rest. The LLMs/agents have actually been doing a stellar job with code reviews. Frankly that’s one area that humans rush through, to the point it’s a running joke that the best way to get a PR granted a “lgtm” is to make it huge. I’ve almost never seen Copilot wave a PR through on the first attempt, but I usually see humans doing that. | ||||||||
| ▲ | distances 4 days ago | parent [-] | |||||||
That smells of bad team practices. Put a practical limit on PRs sizes as the first step, around 500 lines max is a good rule of thumb in my experience. Larger than that, and the expectation then is a number of small PRs to a feature branch. I rarely see a PR that should pass without comments. Your team is being sloppy. | ||||||||
| ||||||||