)]}'
{"zuul/driver/pagure/pagureconnection.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"71d2ae6c585574a49a2e86489f0e06bbb24ec7d1","unresolved":false,"context_lines":[{"line_number":695,"context_line":"            if event.ref and event.ref.startswith(\u0027refs/tags/\u0027):"},{"line_number":696,"context_line":"                change \u003d Tag(project)"},{"line_number":697,"context_line":"                change.tag \u003d event.tag"},{"line_number":698,"context_line":"                change.url \u003d self.getGitwebUrl(project)"},{"line_number":699,"context_line":"            elif event.ref and event.ref.startswith(\u0027refs/heads/\u0027):"},{"line_number":700,"context_line":"                change \u003d Branch(project)"},{"line_number":701,"context_line":"                change.oldrev \u003d event.oldrev"}],"source_content_type":"text/x-python","patch_set":1,"id":"5faad753_0a7d377e","line":698,"updated":"2019-09-12 14:13:18.000000000","message":"We need to supply oldrev and newrev in this case (and also some others).  See https://zuul-ci.org/docs/zuul/user/jobs.html#tag-items","commit_id":"73b31e7453377b1b2b0b67cb283815bd5b3c2ed4"},{"author":{"_account_id":6889,"name":"Fabien Boucher","email":"fboucher@redhat.com","username":"fabien-boucher"},"change_message_id":"7859551578b002bf4dc10a45dcf8fc7febee197d","unresolved":false,"context_lines":[{"line_number":695,"context_line":"            if event.ref and event.ref.startswith(\u0027refs/tags/\u0027):"},{"line_number":696,"context_line":"                change \u003d Tag(project)"},{"line_number":697,"context_line":"                change.tag \u003d event.tag"},{"line_number":698,"context_line":"                change.url \u003d self.getGitwebUrl(project)"},{"line_number":699,"context_line":"            elif event.ref and event.ref.startswith(\u0027refs/heads/\u0027):"},{"line_number":700,"context_line":"                change \u003d Branch(project)"},{"line_number":701,"context_line":"                change.oldrev \u003d event.oldrev"}],"source_content_type":"text/x-python","patch_set":1,"id":"5faad753_22327b5f","line":698,"in_reply_to":"5faad753_0a7d377e","updated":"2019-09-13 19:57:46.000000000","message":"Pagure does not send the newrev through the event so I had to add a merger/executor task to get a commit hash from a ref. Let me know if that\u0027s fine or not.","commit_id":"73b31e7453377b1b2b0b67cb283815bd5b3c2ed4"},{"author":{"_account_id":2,"name":"Monty Taylor","email":"mordred@inaugust.com","username":"mordred"},"change_message_id":"c4f245b1d41548a1205cb1da9506b3a5319d4e13","unresolved":false,"context_lines":[{"line_number":695,"context_line":"            if event.ref and event.ref.startswith(\u0027refs/tags/\u0027):"},{"line_number":696,"context_line":"                change \u003d Tag(project)"},{"line_number":697,"context_line":"                change.tag \u003d event.tag"},{"line_number":698,"context_line":"                change.url \u003d self.getGitwebUrl(project)"},{"line_number":699,"context_line":"            elif event.ref and event.ref.startswith(\u0027refs/heads/\u0027):"},{"line_number":700,"context_line":"                change \u003d Branch(project)"},{"line_number":701,"context_line":"                change.oldrev \u003d event.oldrev"}],"source_content_type":"text/x-python","patch_set":1,"id":"5faad753_808b1066","line":698,"in_reply_to":"5faad753_22327b5f","updated":"2019-09-16 10:52:51.000000000","message":"It seems like an ok solution to me - but obviously interested in what Jim has to say. Can we ask the pagure folks to send the sha in their event in a future release? (I imagine they have it - it\u0027s a shame to have to do the merger job here - but also probably not the WORST thing given frequency)","commit_id":"73b31e7453377b1b2b0b67cb283815bd5b3c2ed4"},{"author":{"_account_id":6889,"name":"Fabien Boucher","email":"fboucher@redhat.com","username":"fabien-boucher"},"change_message_id":"6ee3a5138c52c91367072af73b13803d13991e25","unresolved":false,"context_lines":[{"line_number":695,"context_line":"            if event.ref and event.ref.startswith(\u0027refs/tags/\u0027):"},{"line_number":696,"context_line":"                change \u003d Tag(project)"},{"line_number":697,"context_line":"                change.tag \u003d event.tag"},{"line_number":698,"context_line":"                change.url \u003d self.getGitwebUrl(project)"},{"line_number":699,"context_line":"            elif event.ref and event.ref.startswith(\u0027refs/heads/\u0027):"},{"line_number":700,"context_line":"                change \u003d Branch(project)"},{"line_number":701,"context_line":"                change.oldrev \u003d event.oldrev"}],"source_content_type":"text/x-python","patch_set":1,"id":"5faad753_712d88a6","line":698,"in_reply_to":"5faad753_808b1066","updated":"2019-09-16 13:48:38.000000000","message":"Yes, sure so I\u0027ve opened https://pagure.io/pagure/issue/4610 and tried to propose a fix. If this patch is a no mergeable then I\u0027ll prefer to skip it from the patch chain and have other merged as they already received a +2.","commit_id":"73b31e7453377b1b2b0b67cb283815bd5b3c2ed4"},{"author":{"_account_id":9311,"name":"Tristan Cacqueray","email":"tdecacqu@redhat.com","username":"tristanC"},"change_message_id":"ef0c5e6ffc3077f4d55bafc008f031a1a5b11e40","unresolved":false,"context_lines":[{"line_number":307,"context_line":"        event.project_name \u003d data.get(\u0027project_fullname\u0027)"},{"line_number":308,"context_line":"        event.tag \u003d data.get(\u0027tag\u0027)"},{"line_number":309,"context_line":"        event.ref \u003d \u0027refs/tags/%s\u0027 % event.tag"},{"line_number":310,"context_line":"        # See https://pagure.io/pagure/pull-request/4609"},{"line_number":311,"context_line":"        # Need to wait for the merge in Pagure"},{"line_number":312,"context_line":"        event.oldrev \u003d None"},{"line_number":313,"context_line":"        event.newrev \u003d data.get(\u0027rev\u0027)"},{"line_number":314,"context_line":"        return event"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa7e38b_2a8509ae","line":311,"range":{"start_line":310,"start_character":0,"end_line":311,"end_character":46},"updated":"2019-09-17 13:38:07.000000000","message":"Perhaps move that comment above newrev and mention that until this PR is merged the newrev will be None","commit_id":"e5a4039f705593fb6c48db5a5d99bcc76aa50cfa"},{"author":{"_account_id":16068,"name":"Tobias Henkel","email":"tobias.henkel@bmw.de","username":"tobias.henkel"},"change_message_id":"250b479e37cec8e22a255ae95b60953d9b831ae3","unresolved":false,"context_lines":[{"line_number":205,"context_line":"                self._event_issue_initial_comment,"},{"line_number":206,"context_line":"            \u0027pull-request.tag.added\u0027:"},{"line_number":207,"context_line":"                self._event_pull_request_tags_changed,"},{"line_number":208,"context_line":"            \u0027git.tag.creation\u0027: self._event_tag_created"},{"line_number":209,"context_line":"        }"},{"line_number":210,"context_line":""},{"line_number":211,"context_line":"    def stop(self):"}],"source_content_type":"text/x-python","patch_set":9,"id":"3fa7e38b_2711d405","line":208,"updated":"2019-10-17 13:59:42.000000000","message":"nit: I\u0027d add a trailing comma here to reduce code churn when adding further events","commit_id":"6a34be0ac94562de4a146deff5fdb421782ee8bf"}]}