Merge pull request #1143 from jberry-suse/accept-comment-empty

osclib/stagingapi: accept_status_comment(): avoid making comments for empty projects.
This commit is contained in:
Jimmy Berry 2017-09-20 21:42:01 -05:00 committed by GitHub
commit 605592220b

View File

@ -1568,6 +1568,11 @@ class StagingAPI(object):
self.set_prj_pseudometa(project, meta) self.set_prj_pseudometa(project, meta)
def accept_status_comment(self, project, packages): def accept_status_comment(self, project, packages):
if not len(packages):
# Avoid making accept comments for empty projects which can occur
# when all requests are unselected or something like #1142.
return
# A single comment should be enough to notify everybody, since they are # A single comment should be enough to notify everybody, since they are
# already mentioned in the comments created by select/unselect. # already mentioned in the comments created by select/unselect.
comment = 'Project "{}" accepted. ' \ comment = 'Project "{}" accepted. ' \