Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/hg/template/issue_commit_comment.tmpl

Issue 29344546: Issue 4070 - Correctly handle multiline commit messages in the mercurial hook (Closed)
Patch Set: Handle CRLF line breaks and whitespace in the second line Created May 26, 2016, 9:22 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sitescripts/hg/template/issue_commit_comment.tmpl
===================================================================
--- a/sitescripts/hg/template/issue_commit_comment.tmpl
+++ b/sitescripts/hg/template/issue_commit_comment.tmpl
@@ -1,9 +1,9 @@
{% if changes|length > 1 -%}
Some commits referencing this issue have landed:
{%- else -%}
A commit referencing this issue has landed:
{%- endif %}
{%- for change in changes %}
{% if loop.length > 1 %}- {% endif -%}
-[https://hg.adblockplus.org/{{repository_name}}/rev/{{change.hex()[:12]}} {{change.description()}}]
+[https://hg.adblockplus.org/{{repository_name}}/rev/{{change.hex()[:12]}} {{format_description(change)}}]
{%- endfor -%}

Powered by Google App Engine
This is Rietveld