From 13d76fdfb9f018e40bc3f7ba1381a0843f763d12 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Fri, 18 Oct 2019 08:11:27 +0200 Subject: [PATCH] [FIX] forwardport: the fix it's a per-call parameter not a per-instance one --- forwardport/models/forwardport.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/forwardport/models/forwardport.py b/forwardport/models/forwardport.py index 4f29a48e..9df559c6 100644 --- a/forwardport/models/forwardport.py +++ b/forwardport/models/forwardport.py @@ -132,9 +132,9 @@ class DeleteBranches(models.Model, Queue): _deleter.info('✘ PR owner != FP target owner (%s)', repo_owner) return # probably don't have access to arbitrary repos - github = GH(token=repository.project_id.fp_github_token, repo=fp_remote, check=False) + github = GH(token=repository.project_id.fp_github_token, repo=fp_remote) refurl = 'git/refs/heads/' + branch - ref = github('get', refurl) + ref = github('get', refurl, check=False) if ref.status_code != 200: _deleter.info("✘ branch already deleted (%s)", ref.json()) return @@ -149,7 +149,7 @@ class DeleteBranches(models.Model, Queue): ) return - r = github('delete', refurl) + r = github('delete', refurl, check=False) assert r.status_code == 204, \ "Tried to delete branch %s of %s, got %s" % ( branch, self.pr_id.display_name,