ReviewBot: use super().check_source_submission() in subclasses.
No need to duplicate the log message in multiple places.
This commit is contained in:
parent
6eefb429d3
commit
92a5f60369
@ -62,7 +62,7 @@ class FactorySourceChecker(ReviewBot.ReviewBot):
|
|||||||
['source', prj, '00Meta', 'lookup.yml']))
|
['source', prj, '00Meta', 'lookup.yml']))
|
||||||
|
|
||||||
def check_source_submission(self, src_project, src_package, src_rev, target_project, target_package):
|
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)
|
src_srcinfo = self.get_sourceinfo(src_project, src_package, src_rev)
|
||||||
if src_srcinfo is None:
|
if src_srcinfo is None:
|
||||||
# source package does not exist?
|
# source package does not exist?
|
||||||
|
@ -135,7 +135,7 @@ class Leaper(ReviewBot.ReviewBot):
|
|||||||
}
|
}
|
||||||
|
|
||||||
def check_source_submission(self, src_project, src_package, src_rev, target_project, target_package):
|
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)
|
src_srcinfo = self.get_sourceinfo(src_project, src_package, src_rev)
|
||||||
package = target_package
|
package = target_package
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user