Dzahn has submitted this change and it was merged.
Change subject: Strip "(N comments)" lines from IRC output
......................................................................
Strip "(N comments)" lines from IRC output
We want the cover message, not metadata
Change-Id: Ia40a64f05e12336ae2525ec0b7521ec095b42782
---
M files/gerrit/hooks/comment-added
1 file changed, 2 insertions(+), 1 deletion(-)
Approvals:
jenkins-bot: Verified
Dzahn: Looks good to me, approved
diff --git a/files/gerrit/hooks/comment-added b/files/gerrit/hooks/comment-added
index 57b9eed..fbc9cab 100755
--- a/files/gerrit/hooks/comment-added
+++ b/files/gerrit/hooks/comment-added
@@ -20,7 +20,8 @@
(options, args) = self.parser.parse_args()
if options.draft == "true":
return # Don't report drafts to IRC!
- comment = re.sub(r"^\s*Patch Set \d+:.*$", '', options.comment,
flags=re.MULTILINE).strip().splitlines()
+ comment = re.sub(r"^\s*Patch Set \d+:.*$", '', options.comment,
flags=re.MULTILINE)
+ comment = re.sub(r"^\s*\(\d+ comments\).*$", '', comment,
flags=re.MULTILINE).strip().splitlines()
if comment:
comment = comment[0]
if len(comment) > 103:
--
To view, visit https://gerrit.wikimedia.org/r/60317
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia40a64f05e12336ae2525ec0b7521ec095b42782
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Demon <ch...@wikimedia.org>
Gerrit-Reviewer: Dzahn <dz...@wikimedia.org>
Gerrit-Reviewer: Ryan Lane <rl...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
