From 92a5f6036948852053dd1754b2779748619c699e Mon Sep 17 00:00:00 2001 From: Jimmy Berry Date: Fri, 10 Feb 2017 16:09:51 -0600 Subject: [PATCH] ReviewBot: use super().check_source_submission() in subclasses. No need to duplicate the log message in multiple places. --- check_source_in_factory.py | 2 +- leaper.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/check_source_in_factory.py b/check_source_in_factory.py index d5ebfcf2..9b7dc7f7 100755 --- a/check_source_in_factory.py +++ b/check_source_in_factory.py @@ -62,7 +62,7 @@ class FactorySourceChecker(ReviewBot.ReviewBot): ['source', prj, '00Meta', 'lookup.yml'])) def check_source_submission(self, src_project, src_package, src_rev, target_project, target_package): - self.logger.info("%s/%s@%s -> %s/%s"%(src_project, src_package, src_rev, target_project, target_package)) + super(FactorySourceChecker, self).check_source_submission(src_project, src_package, src_rev, target_project, target_package) src_srcinfo = self.get_sourceinfo(src_project, src_package, src_rev) if src_srcinfo is None: # source package does not exist? diff --git a/leaper.py b/leaper.py index 5d0e3482..44854dd2 100755 --- a/leaper.py +++ b/leaper.py @@ -135,7 +135,7 @@ class Leaper(ReviewBot.ReviewBot): } def check_source_submission(self, src_project, src_package, src_rev, target_project, target_package): - self.logger.info("%s/%s@%s -> %s/%s"%(src_project, src_package, src_rev, target_project, target_package)) + super(Leaper, self).check_source_submission(src_project, src_package, src_rev, target_project, target_package) src_srcinfo = self.get_sourceinfo(src_project, src_package, src_rev) package = target_package