3 ms·
The content is the diff (including description/test plan), which I'd argue should be the name. It could be in a draft state, awaiting review, accepted for merg
by gibolt 4y ago
The content is the diff (including description/test plan), which I'd argue should be the name.
It could be in a draft state, awaiting review, accepted for merge, or being merged.
'diff' is the shortest at most descriptive name.
- erik_seaberg 4y agoWhen I hear “PR” I think of a tree of git commits. When I hear “diff” I think of one set of squashed text changes, for a tool that doesn’t support git.
- cjtrowbridge 4y agoThere is no "correct" name. You only see things from your standpoint, and so does everyone else. There is nothing about your opinion that is any less arbitrary than all the opinions that disagree with you.