3 ms·
This currently is my team's only AI-use policy and I thik it's working out fantastically. "No AI PR Descriptions" is a great rule because it does 3 things: 1.
by hannasanarion 8d ago
This currently is my team's only AI-use policy and I thik it's working out fantastically.
"No AI PR Descriptions" is a great rule because it does 3 things:
1. It's a hard binary that's easy to recognize and enforce
2. It establishes personal ownership for the submitter, making it psychologically difficult to submit something you don't understand. If a person has to describe what a change does, how, and why, then they must have looked at it and made a real attempt to understand it, because you can't map a territory you have never seen. This alone prevents "claude-code run amok" scenarios where devs have completely abdicated responsibility that are a real pain to clean up.
3. It informs the reviewer of the change's intent. A human PR tells reviewers what a thing is intended to do, as well as the authors belief about what it does, which they can use as a framework to critique the actual substance. It becomes easier to notice missed edge cases, code behavior that subtly differs from intent, or changes that would block off or complicate an intended future project direction or reduce maintainability.
And as a bonus, at time of writing, AI pr descriptions basically always suck. They are usually full of irrelevant implementation details, misplaced emphasis, weird presuppositions that seem to come out of nowhere and are really annoying to read, etc. That won't be true forever, they are always getting better, but it is true now, so that's a temporary 4th benefit.
- vova_hn2 7d agoGreat comment, no idea why it was flagged
- dawnerd 6d agoOur rules are: short title that should match the task in the ticket tracker, link to the task in the body as the first item, pr summary that’s human readable, screenshots of any front end changes at all the breakpoints. The screenshot requirement is huge, it’s caught a lot of people skipping visual testing locally. Even if they have an llm generate the screenshot we know it’s been at least tested by something and allows for a quick check before someone even wastes their time looking at code.