[MediaWiki-commits] [Gerrit] labs...wikibugs2[master]: Ignore @Phabricator_maintenance and clean up code

2016-07-28 Thread jenkins-bot (Code Review) 
jenkins-bot has submitted this change and it was merged.

Change subject: Ignore @Phabricator_maintenance and clean up code
..


Ignore @Phabricator_maintenance and clean up code

Bug: T141570
Change-Id: If4d13437ee594de2993d7e49aad01eede2d1b3f6
---
M redis2irc.py
1 file changed, 4 insertions(+), 7 deletions(-)

Approvals:
  Legoktm: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/redis2irc.py b/redis2irc.py
index d74dcff..4e654aa 100644
--- a/redis2irc.py
+++ b/redis2irc.py
@@ -78,13 +78,10 @@
 :type bot: Redis2Irc
 :type useful_info: dict
 """
-if useful_info['user'] in ('gerritbot', 'ReleaseTaggerBot'):
-# Ignore "Patch to review" stuff or deploy tagging
-logger.debug("Skipped %s by gerritbot/ReleaseTaggerBot" % 
useful_info['url'])
-return
-if useful_info['user'] == 'Stashbot':
-# Ignore Stashbot's comments about SAL messages
-logger.debug("Skipped %s by Stashbot" % useful_info['url'])
+ignored = ('gerritbot', 'ReleaseTaggerBot', 'Stashbot', 
'Phabricator_maintenance')
+if useful_info['user'] in ignored:
+# Ignore some Phabricator bots
+logger.debug("Skipped %(url)s by %(user)s" % useful_info)
 return
 updated = bot.chanfilter.update()
 if updated:

-- 
To view, visit https://gerrit.wikimedia.org/r/301742
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If4d13437ee594de2993d7e49aad01eede2d1b3f6
Gerrit-PatchSet: 1
Gerrit-Project: labs/tools/wikibugs2
Gerrit-Branch: master
Gerrit-Owner: Legoktm 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] labs...wikibugs2[master]: Ignore @Phabricator_maintenance and clean up code

2016-07-28 Thread Legoktm (Code Review) 
Legoktm has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/301742

Change subject: Ignore @Phabricator_maintenance and clean up code
..

Ignore @Phabricator_maintenance and clean up code

Bug: T141570
Change-Id: If4d13437ee594de2993d7e49aad01eede2d1b3f6
---
M redis2irc.py
1 file changed, 4 insertions(+), 7 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/labs/tools/wikibugs2 
refs/changes/42/301742/1

diff --git a/redis2irc.py b/redis2irc.py
index d74dcff..4e654aa 100644
--- a/redis2irc.py
+++ b/redis2irc.py
@@ -78,13 +78,10 @@
 :type bot: Redis2Irc
 :type useful_info: dict
 """
-if useful_info['user'] in ('gerritbot', 'ReleaseTaggerBot'):
-# Ignore "Patch to review" stuff or deploy tagging
-logger.debug("Skipped %s by gerritbot/ReleaseTaggerBot" % 
useful_info['url'])
-return
-if useful_info['user'] == 'Stashbot':
-# Ignore Stashbot's comments about SAL messages
-logger.debug("Skipped %s by Stashbot" % useful_info['url'])
+ignored = ('gerritbot', 'ReleaseTaggerBot', 'Stashbot', 
'Phabricator_maintenance')
+if useful_info['user'] in ignored:
+# Ignore some Phabricator bots
+logger.debug("Skipped %(url)s by %(user)s" % useful_info)
 return
 updated = bot.chanfilter.update()
 if updated:

-- 
To view, visit https://gerrit.wikimedia.org/r/301742
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If4d13437ee594de2993d7e49aad01eede2d1b3f6
Gerrit-PatchSet: 1
Gerrit-Project: labs/tools/wikibugs2
Gerrit-Branch: master
Gerrit-Owner: Legoktm 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits