)]}'
{"zuul/manager/__init__.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"f78b674ff7310581c61e91d55f87686af18a4f3c","unresolved":false,"context_lines":[{"line_number":1846,"context_line":"    def _reportItem(self, item):"},{"line_number":1847,"context_line":"        log \u003d get_annotated_logger(self.log, item.event)"},{"line_number":1848,"context_line":"        log.debug(\"Reporting change %s\", item.change)"},{"line_number":1849,"context_line":"        ret \u003d True  # Means error as returned by trigger.report"},{"line_number":1850,"context_line":""},{"line_number":1851,"context_line":"        # In the case of failure, we may not have completed an initial"},{"line_number":1852,"context_line":"        # merge which would get the layout for this item, so in order"}],"source_content_type":"text/x-python","patch_set":1,"id":"10b35ca5_aacdc032","line":1849,"updated":"2022-03-02 00:39:38.000000000","message":"I do too, and I actually started to unwind that in this change, but it became too much to deal with in this change so I tried to keep this change minimal for review.","commit_id":"370d36c0257b5849c0b08308073b690b16b2a180"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"30562d84c156b6f9050f36c3551c9a2f7ea0d5f6","unresolved":true,"context_lines":[{"line_number":1846,"context_line":"    def _reportItem(self, item):"},{"line_number":1847,"context_line":"        log \u003d get_annotated_logger(self.log, item.event)"},{"line_number":1848,"context_line":"        log.debug(\"Reporting change %s\", item.change)"},{"line_number":1849,"context_line":"        ret \u003d True  # Means error as returned by trigger.report"},{"line_number":1850,"context_line":""},{"line_number":1851,"context_line":"        # In the case of failure, we may not have completed an initial"},{"line_number":1852,"context_line":"        # merge which would get the layout for this item, so in order"}],"source_content_type":"text/x-python","patch_set":1,"id":"78578ab6_abfdcaf6","line":1849,"updated":"2022-03-01 23:57:46.000000000","message":"Nit: I find it confusing to carry a ret return value then negate it when we actually return it. Maybe we should call this reported instead? and then track it here with that in mind?","commit_id":"370d36c0257b5849c0b08308073b690b16b2a180"}]}