)]}'
{"openstack/clustering/v1/_proxy.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"4deae915573d763242a8cb48dd88417b9b1bff66","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :returns: ``None``"},{"line_number":42,"context_line":"        \"\"\""},{"line_number":43,"context_line":"        self._delete(cluster.Cluster, value, ignore_missing)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"    def find_cluster(self, name_or_id):"},{"line_number":46,"context_line":"        \"\"\"Find a single cluster."}],"source_content_type":"text/x-python","patch_set":6,"id":"5afe65bd_5fd5eab6","line":43,"updated":"2015-06-03 19:47:10.000000000","message":"Ah, this should be ignore_missing\u003dignore_missing","commit_id":"d3014928d8a73e2b45ae7992c5be6e8ffab33297"}],"openstack/clustering/v1/cluster.py":[{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"d63629eedf02758753c75dfa55aa8f49dfcc2e43","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    allow_list \u003d True"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"    # Properties"},{"line_number":32,"context_line":"    name \u003d resource.prop(\u0027name\u0027)"},{"line_number":33,"context_line":"    profile_id \u003d resource.prop(\u0027profile_id\u0027)"},{"line_number":34,"context_line":"    user \u003d resource.prop(\u0027user\u0027)"},{"line_number":35,"context_line":"    project \u003d resource.prop(\u0027project\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a016987_d184485e","line":32,"updated":"2015-05-27 18:12:23.000000000","message":"Could you add docstrings above each of these properties so we can have API documentation generated for this?","commit_id":"e621df20b9467e82d770a69be050305c1c86f778"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"d63629eedf02758753c75dfa55aa8f49dfcc2e43","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        }"},{"line_number":65,"context_line":"        return self.action(session, body)"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def del_nodes(self, session, nodes):"},{"line_number":68,"context_line":"        body \u003d {"},{"line_number":69,"context_line":"            \u0027del_nodes\u0027: {"},{"line_number":70,"context_line":"                \u0027nodes\u0027: nodes,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a016987_31e71c42","line":67,"updated":"2015-05-27 18:12:23.000000000","message":"I\u0027d prefer if this was delete_nodes so it\u0027s a bit more clear - doesn\u0027t have to match the del_nodes sent in the body.","commit_id":"e621df20b9467e82d770a69be050305c1c86f778"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"d63629eedf02758753c75dfa55aa8f49dfcc2e43","unresolved":false,"context_lines":[{"line_number":146,"context_line":"        }"},{"line_number":147,"context_line":"        return self.action(session, body)"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    def to_dict(self):"},{"line_number":150,"context_line":"        info \u003d {"},{"line_number":151,"context_line":"            \u0027id\u0027: self.id,"},{"line_number":152,"context_line":"            \u0027name\u0027: self.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a016987_d139e8a4","line":149,"updated":"2015-05-27 18:12:23.000000000","message":"What\u0027s the use case here? I think we could add something like this to the base Resource class instead of in a one-off subclass.","commit_id":"e621df20b9467e82d770a69be050305c1c86f778"},{"author":{"_account_id":8246,"name":"Qiming Teng","email":"tengqm@outlook.com","username":"tengqm"},"change_message_id":"efba6f5eb096bdeb7fdc72f8614ed4f6c0100d56","unresolved":false,"context_lines":[{"line_number":146,"context_line":"        }"},{"line_number":147,"context_line":"        return self.action(session, body)"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"    def to_dict(self):"},{"line_number":150,"context_line":"        info \u003d {"},{"line_number":151,"context_line":"            \u0027id\u0027: self.id,"},{"line_number":152,"context_line":"            \u0027name\u0027: self.name,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a016987_e6609a37","line":149,"in_reply_to":"7a016987_d139e8a4","updated":"2015-05-28 01:46:04.000000000","message":"It would be great having base Resource class to do this. The use case was to feed the returned dict to command line formatting (yaml, pretty-table, json) etc.","commit_id":"e621df20b9467e82d770a69be050305c1c86f778"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"ed339d61cd4a14d4826817bff7587be76f965c94","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        }"},{"line_number":167,"context_line":"        return self.action(session, body)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"    def to_dict(self):"},{"line_number":170,"context_line":"        info \u003d {"},{"line_number":171,"context_line":"            \u0027id\u0027: self.id,"},{"line_number":172,"context_line":"            \u0027name\u0027: self.name,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a016987_a1572c01","line":169,"updated":"2015-05-28 02:01:47.000000000","message":"Everything else looks good in these last changesets, but if you remove this method and we put this type of change in base Resource separately, it\u0027s a +2. Doing this in Resource should be fairly easy since we have the _attrs dict in there already, so just need to make some kind of public accessor to it.","commit_id":"014b1148eaa1f5ebcd0a229e0b497c521ceeb3e9"},{"author":{"_account_id":8246,"name":"Qiming Teng","email":"tengqm@outlook.com","username":"tengqm"},"change_message_id":"02d2959584659e2f77f3377f471f390f07c207ca","unresolved":false,"context_lines":[{"line_number":166,"context_line":"        }"},{"line_number":167,"context_line":"        return self.action(session, body)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"    def to_dict(self):"},{"line_number":170,"context_line":"        info \u003d {"},{"line_number":171,"context_line":"            \u0027id\u0027: self.id,"},{"line_number":172,"context_line":"            \u0027name\u0027: self.name,"}],"source_content_type":"text/x-python","patch_set":5,"id":"7a016987_6197f490","line":169,"in_reply_to":"7a016987_a1572c01","updated":"2015-05-28 02:06:21.000000000","message":"Cool. Will remove this function and propose a patch to the Resource class.","commit_id":"014b1148eaa1f5ebcd0a229e0b497c521ceeb3e9"}]}