)]}'
{"zuul/driver/github/githubreporter.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"c50d1567b8f2b8b19477ba8e690835b1772d820a","unresolved":false,"context_lines":[{"line_number":192,"context_line":""},{"line_number":193,"context_line":"        return message"},{"line_number":194,"context_line":""},{"line_number":195,"context_line":"    def getSubmitAllowNeeds(self, tenant_name, pipeline_name):"},{"line_number":196,"context_line":"        \"\"\"Get a list of code review labels that are allowed to be"},{"line_number":197,"context_line":"        \"needed\" in the submit records for a change, with respect"},{"line_number":198,"context_line":"        to this queue.  In other words, the list of review labels"}],"source_content_type":"text/x-python","patch_set":8,"id":"5f7c97a3_8e7d5de8","line":195,"updated":"2018-06-01 21:03:27.000000000","message":"A reporter is instantiated for every pipeline (which is unique to a tenant).  So we shouldn\u0027t need to pass both of these in.  At the least, we can just pass in the pipeline and get to the rest of the info from there.\n\nBut since the reporter itself is attached to the pipeline, it feels like we shouldn\u0027t need to pass anything here.  How about instead we update the reporter constructor and pass in the pipeline when we construct it?  Then in here, we can use self.pipeline.\n\nIn fact, if we do that, we can go ahead and set the status string we use in the initializer for this, and use that pre-computed value here and in the report method.","commit_id":"1a54a47c3ae8d02f72430239681c3d568a8ffb93"}]}