Rush has submitted this change and it was merged. Change subject: phabricator: direct task messaging matching ......................................................................
phabricator: direct task messaging matching we have a some vendors who cc tasks directly and whether a message is directed at a task is determined by regex. This is more liberal to make up for the plethera of insanity that is vendor email client behaviors. Bug: T117113 Change-Id: Id585d5daab48b5b02518b6521db1084bf4c91285 --- M modules/phabricator/files/phab_epipe.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Rush: Looks good to me, approved RobH: Looks good to me, but someone else must approve jenkins-bot: Verified diff --git a/modules/phabricator/files/phab_epipe.py b/modules/phabricator/files/phab_epipe.py index 6afcf07..8432a82 100644 --- a/modules/phabricator/files/phab_epipe.py +++ b/modules/phabricator/files/phab_epipe.py @@ -60,7 +60,7 @@ :returns: string """ for dest in [m.strip() for m in list_of_dests]: - task = re.match('^[T|t](\d+)@', dest) + task = re.search('[T|t](\d+)@phabricator', dest) if task: return int(task.group(1)) return '' -- To view, visit https://gerrit.wikimedia.org/r/256995 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id585d5daab48b5b02518b6521db1084bf4c91285 Gerrit-PatchSet: 3 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Rush <r...@wikimedia.org> Gerrit-Reviewer: RobH <r...@wikimedia.org> Gerrit-Reviewer: Rush <r...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits