[IMP] runbot_merge: only label PR if not created merged/closed

Would generate a ton of GH API churn when synchronising.
This commit is contained in:
Xavier Morel 2018-04-03 15:02:59 +02:00 committed by xmo-odoo
parent 39a0d723af
commit 47428a4580

View File

@ -559,12 +559,13 @@ class PullRequests(models.Model):
@api.model @api.model
def create(self, vals): def create(self, vals):
pr = super().create(vals) pr = super().create(vals)
self.env['runbot_merge.pull_requests.tagging'].create({ if pr.state not in ('closed', 'merged'):
'pull_request': pr.number, self.env['runbot_merge.pull_requests.tagging'].create({
'repository': pr.repository.id, 'pull_request': pr.number,
'state_from': False, 'repository': pr.repository.id,
'state_to': pr._tagstate, 'state_from': False,
}) 'state_to': pr._tagstate,
})
return pr return pr
@api.multi @api.multi