Extract patchinfo check into a function

This commit is contained in:
Stephan Kulow 2017-04-28 12:23:08 +02:00 committed by Ondřej Súkup
parent 93b8bfbd3f
commit 9a418f780f
No known key found for this signature in database
GPG Key ID: 18E6D11A219AF3B7

View File

@ -223,10 +223,14 @@ class ReviewBot(object):
overall = ret overall = ret
return overall return overall
@staticmethod
def _is_patchinfo(pkgname):
return pkgname == 'patchinfo' or pkgname.startswith('patchinfo.')
def check_action_maintenance_incident(self, req, a): def check_action_maintenance_incident(self, req, a):
dst_package = a.src_package dst_package = a.src_package
# Ignoring patchinfo package for checking # Ignoring patchinfo package for checking
if a.src_package == 'patchinfo' or a.src_package.startswith('patchinfo.'): if self._is_patchinfo(a.src_package):
self.logger.info("package is patchinfo, ignoring") self.logger.info("package is patchinfo, ignoring")
return None return None
# dirty obs crap # dirty obs crap
@ -238,7 +242,7 @@ class ReviewBot(object):
def check_action_maintenance_release(self, req, a): def check_action_maintenance_release(self, req, a):
pkgname = a.src_package pkgname = a.src_package
if pkgname == 'patchinfo' or pkgname.startswith('patchinfo.'): if self._is_patchinfo(pkgname):
return None return None
linkpkg = self._get_linktarget_self(a.src_project, pkgname) linkpkg = self._get_linktarget_self(a.src_project, pkgname)
if linkpkg is not None: if linkpkg is not None: