Remix.run Logo
morkalork 6 hours ago

I am so very tired of 2 page long PR descriptions for a 5 line change.

flemhans 16 minutes ago | parent | next [-]

Use your existing desired style as exemplars

drooby 3 hours ago | parent | prev | next [-]

My team solved this by creating a PR draft skill that clamps the length of the description to 3-5 sentences max. Those 3-5 sentences must only say WHAT is changing and WHY.

I find it to be far more useful than when humans wrote PR descriptions. Many engineers didn't write one, and those that did were poorly written... this problem is mostly solved for us.. it still has LLMism speak.. but it's useful enough for me to get the context I need to do my review.

AlotOfReading 2 hours ago | parent [-]

The challenge of hard rules like this is that they're always overly restrictive. I've made multi-thousand file PRs that needed two lines of description including the title, and 5 line PRs that needed a 1hr presentation to fully explain them.

My personal guideline is that writing for humans should be done by humans.

baq 5 hours ago | parent | prev [-]

Some 5-line changes deserve a phd.

But yeah, most probably don’t.