check_tags: use len() instead of list() since only count is of interest.

2017-02-09 21:42:06 -06:00 · 2017-02-09 21:42:06 -06:00 · 7ddd4287e1
commit 7ddd4287e1
parent c1fde90c64
1 changed files with 2 additions and 2 deletions
--- 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