#6125 svn repository commit emails should include revision number

v1.0.0
closed
nobody
42cc (432)
Tracker
2015-08-20
2013-04-17
Anonymous
No

Originally created by: zeromus

Here's the subject of a notification email for a commit to svn repository:

[fceultra:code] zeromus committed to FCEUX Code: add missing mmc5 savestate variables

I would like to see something like

[fceultra:code] r2928 - zeromus committed: add missing mmc5 savestate variables

the "FCEUX Code" is pretty redundant (not sure why its named differently than [fceultra:code] but that's our problem I think). The revision number is very important. It's contained in the body of the email as a url, but thats not as convenient.

Related

Tickets: #6125

Discussion

  • Dave Brondsema

    Dave Brondsema - 2013-05-01
    • labels: --> 42cc
    • Milestone: limbo --> forge-backlog
     
  • Dave Brondsema

    Dave Brondsema - 2013-05-01

    I agree, makes sense to remove the redundancy and add the revision number. Note that this same template is used for hg & git which can have multiple revisions pushed at the same time, so in that case a revision can't be shown.

    Another small tweak to the body of this email -- each commit in the body is like "branch-name : message" but on SVN there isn't a branch name so the colon is weird and not needed. Remove it if there isn't a branch name.

     
  • Igor Bondarenko - 2013-05-02
    • status: open --> in-progress
     
  • Igor Bondarenko - 2013-05-02

    Created #342: [#6125] svn repository commit emails should include revision number (2cp)

     

    Related

    Tickets: #6125

  • Igor Bondarenko - 2013-05-17
    • status: in-progress --> code-review
     
  • Igor Bondarenko - 2013-05-17

    Closed #342. je/42cc_6125

     
    • QA: Tim Van Steenburgh
    • Milestone: forge-backlog --> forge-may-31
     
    • status: code-review --> closed
     

Log in to post a comment.