From 7ddd4287e1be75c1ae71dd3dde3c30d67ae3c24b Mon Sep 17 00:00:00 2001 From: Jimmy Berry Date: Thu, 9 Feb 2017 21:42:06 -0600 Subject: [PATCH] check_tags: use len() instead of list() since only count is of interest. --- check_tags_in_requests.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/check_tags_in_requests.py b/check_tags_in_requests.py index f96a2eb5..efd0c5f0 100755 --- a/check_tags_in_requests.py +++ b/check_tags_in_requests.py @@ -96,8 +96,8 @@ by OBS on which this bot relies on. return None xml = ET.parse(f) - has_changes = list(xml.findall('./issues/issue')) - if not has_changes: + issues = len(xml.findall('./issues/issue')) + if issues == 0: self.logger.debug("reject: diff contains no tags") return False return True