• DontTakeMySky@lemmy.world
    link
    fedilink
    arrow-up
    10
    ·
    1 year ago

    Don’t just summarize the content though, summarize the rationale or how things connect. I can read your diff myself to see what changed, I want to know the logical connections, the reason you did X and not Y, etc.

    Or just say “stuff” and provide that context in the PR description separately, no need to overdo the commit log on a feature branch if you’re using squash merges from your PR.