Page MenuHomePhabricator

gerrit-wm: Include the subject of the commit in notifications about reviews/comments in the irc messages
Closed, DeclinedPublic


<gerrit-wm> New review: Santhosh; "(no comment)" [mediawiki/extensions/Translate] (master); V: 1 C: 2; -

A line of commit summary can be nice.

Version: unspecified
Severity: normal



Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 12:17 AM
bzimport added a project: Gerrit.
bzimport set Reference to bz35948.
bzimport added a subscriber: Unknown Object (MLST).

Still valid. Works for patchsets though:

<gerrit-wm\x08>\x08\x0f New patchset: Jeroen De Dauw; "Fix setup script for MySQL" [mediawiki/extensions/SemanticMediaWiki] (master) -
\x03\x08<\x08gerrit-wm\x08>\x08\x0f New review: Jeroen De Dauw; "Patch Set 1: Verified+2 Code-Review+2" [mediawiki/extensions/SemanticMediaWiki] (master); V: 2 C: 2; -
\x03\x08<\x08gerrit-wm\x08>\x08\x0f Change merged: Jeroen De Dauw; [mediawiki/extensions/SemanticMediaWiki] (master) -

We deployed a brand new gerrit-wm replacement (currently named lolrrit-wm, will be named gerrit-wm at some point). Should be more helpful in these cases, can you please take a look and confirm?