Merge pull request #70 from Centril/rollup-made-fix

pietroalbini · web-flow · commit aee959305e43 · 2019-10-07T09:02:36.000+02:00
Fix PR number in ROLLUP_MADE event.
diff --git a/homu/server.py b/homu/server.py
@@ -337,6 +337,8 @@ def rollup(user_gh, state, repo_label, repo_cfg, repo):
             user_repo.owner.login + ':' + branch_name,
             body,
         )
+        state.num = pull.number
+        state.head_sha = pull.head.sha
         state.change_labels(LabelEvent.ROLLUP_MADE)
     except github3.models.GitHubError as e:
         return e.response.text

Original file line number	Diff line number	Diff line change
`@@ -337,6 +337,8 @@ def rollup(user_gh, state, repo_label, repo_cfg, repo):`
`337`	`337`	`user_repo.owner.login + ':' + branch_name,`
`338`	`338`	`body,`
`339`	`339`	`)`
	`340`	`+ state.num = pull.number`
	`341`	`+ state.head_sha = pull.head.sha`
`340`	`342`	`state.change_labels(LabelEvent.ROLLUP_MADE)`
`341`	`343`	`except github3.models.GitHubError as e:`
`342`	`344`	`return e.response.text`