· Föderation EN Sa 24.05.2025 02:43:34 @Lucomo Regardless of tools (IDEs, StackOverflow, coding tools), I think authors of PRs are responsible for PR contents. And the burden of proof that a change is safe and desirable is on the author, not the reviewer. That's my current opinion, anyways :) |
Föderation EN Sa 24.05.2025 05:35:58 |
Föderation EN Sa 24.05.2025 06:11:07 @adamsnotes @Lucomo What I'm saying is that, if a reviewer is confronted with a 5k line change, they should feel empowered to reject it until the author can convince them that it's safe and that it's the minimal change necessary. Big projects like Kubernetes do have big PRs sometimes, but I thinkthey are from people who have a reputation in the project and thus a high degree of trust. I imagine that a lot are simply rejected without much review if they come from new contributors |
Föderation EN Sa 24.05.2025 07:05:36 |