)]}'
{"test/unit/__init__.py":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"b9480386b95101ad17437af5e6f2692673f76ec7","unresolved":false,"context_lines":[{"line_number":1060,"context_line":"        finally:"},{"line_number":1061,"context_line":"            # A FakeStatus may raise an exception during the FakeConn.__init__"},{"line_number":1062,"context_line":"            # but we always want to call give_connect..."},{"line_number":1063,"context_line":"            if \u0027give_connect\u0027 in kwargs:"},{"line_number":1064,"context_line":"                give_conn_fn \u003d kwargs[\u0027give_connect\u0027]"},{"line_number":1065,"context_line":"                argspec \u003d inspect.getargspec(give_conn_fn)"},{"line_number":1066,"context_line":"                if argspec.keywords or \u0027connection_id\u0027 in argspec.args:"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_dd7c23b8","line":1063,"updated":"2018-06-29 17:18:55.000000000","message":"I would prefer  give_conn_fn \u003d kwargs.get(\u0027give_connect\u0027), then if give_conn_fn, purely in case someone passes a dict later, formed like req is at like 1093 below (where you have connection_id set to None, instead of not set at all).","commit_id":"67c0fc210b1816f3327114c9cbcc0feb866e6e68"}]}