Opened 4 years ago

Closed 4 years ago

#17334 closed Cleanup/optimization (fixed)

Document the commit message style for backports

Reported by: aaugustin Owned by: krzysiumed
Component: Documentation Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

As discussed in #django-dev, it seems that the appropriate style is:

[1.3.X] Fixed #**** -- Frobnicated spam around. Backport of r*****.

Attachments (2)

17334.diff (972 bytes) - added by krzysiumed 4 years ago.
17334_v2.diff (1.1 KB) - added by krzysiumed 4 years ago.

Download all attachments as: .zip

Change History (6)

Changed 4 years ago by krzysiumed

comment:1 Changed 4 years ago by krzysiumed

  • Has patch set
  • Owner changed from nobody to krzysiumed
  • Status changed from new to assigned

comment:2 Changed 4 years ago by ramiro

  • Patch needs improvement set

Why do the proposed patch remove the following text?

Write detailed commit messages in the past tense, not present tense.
* Good: "Fixed Unicode bug in RSS API." 
* Bad: "Fixes Unicode bug in RSS API." 
* Bad: "Fixing Unicode bug in RSS API." 

I'm not aware of any discussion and agreement about that requirement being changed or relaxed.

Changed 4 years ago by krzysiumed

comment:3 Changed 4 years ago by krzysiumed

  • Patch needs improvement unset

Sorry for this stupid mistake. The pattern of commit messages is in past tense, so I thought the problem of tenses disappeared. I didn't notice that the problem actually did not disappeare because the pattern doesn't mention tense of description.

comment:4 Changed 4 years ago by timo

  • Resolution set to fixed
  • Status changed from assigned to closed

In [17294]:

Fixed #17334 - Documented the commit message style for backports; thanks krzysiumed for the draft patch.

Note: See TracTickets for help on using tickets.
Back to Top