3 ms·
> That is less useful when the changes are editing the tests but we don't know if a human has validated the assertions. Yep. I've witnessed this first hand man
by icedchai 4mo ago
> That is less useful when the changes are editing the tests but we don't know if a human has validated the assertions.
Yep. I've witnessed this first hand many times. AI-enthusiastic coworker submits a PR. The tests don't pass. "Can you fix the tests? Then I'll review."
Next commit has `assert status == 200` changed to `assert status == 500` all over the place, among other things. Yes, technically, the tests now pass, but...
Last summer, this went on with one guy for weeks. Thousands and thousands of lines of slop. Eventually he was moved off the project and we threw away all his changes.
- gravypod 4mo agoI fear what's going to happen with less tech literate managers who don't understand these cases.
- icedchai 4mo agoYep. It's bad enough as it is! I've had several tech illiterate managers that could not review a PR to save their life. They have zero understanding of the work product. Manager first, engineering second.