Our 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, s

Our 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.