from Hacker News

Ask HN: Pull request body how detailed should it be?

by crtlaltdel on 1/23/20, 10:00 PM with 2 comments

Our team has the most inconsistent content for pull request body messages I have ever seen. Some will drop a link to a ticket without any other context, some will write a little paragraph that describes the intent of the changes, and still others put the ticket number in their pull request title and leave the body blank.

Personally, I enjoy the small narratives. The tickets are usually void of useful technical specifics (beyond maybe what component should be worked on).

So, my questions to HN:

1. How detailed should one make a pull request body?

2.Is there any value in pushing the point? We have lots to do and limited bandwidth, both points the CTO has tossed at me when I raise issues like this.

EDIT: format

  • by Ahmedb on 1/23/20, 10:12 PM

    All PRs should have a standard format it helps both the reviewers and the author.

    One thing that works well is to use prefilled template with 3/4 main sections.

    Something like:

    ### What 'Describe your change'

    ### Why `Describe what drove this change (bugfix, feature, crash,...)`

    ### Links 'Link to ticket or design`

    ### Testing `What should reviewers look for`