)]}'
{"keystoneclient/v3/regions.py":[{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"7159858866bc3d25fc21ac2fbabf50f8142860ea","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    Attributes:"},{"line_number":20,"context_line":"        * id: a string that identifies the region."},{"line_number":21,"context_line":"        * description: a string that describes the region."},{"line_number":22,"context_line":"        * parent_region_id: a pre-existing region in the backend or its id"},{"line_number":23,"context_line":"                            field. Allows for hierarchical region organization."},{"line_number":24,"context_line":"        * enabled: determines whether the endpoint appears in the catalog."},{"line_number":25,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_7bce102f","line":22,"range":{"start_line":22,"start_character":72,"end_line":22,"end_character":74},"updated":"2016-07-06 12:32:03.000000000","message":"(nit) ID","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"8d0a03826bb5620f30c9e5b53b985ada0bff378f","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    Attributes:"},{"line_number":20,"context_line":"        * id: a string that identifies the region."},{"line_number":21,"context_line":"        * description: a string that describes the region."},{"line_number":22,"context_line":"        * parent_region_id: a pre-existing region in the backend or its id"},{"line_number":23,"context_line":"                            field. Allows for hierarchical region organization."},{"line_number":24,"context_line":"        * enabled: determines whether the endpoint appears in the catalog."},{"line_number":25,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_9aa615ac","line":22,"range":{"start_line":22,"start_character":72,"end_line":22,"end_character":74},"in_reply_to":"1aa78d24_7bce102f","updated":"2016-07-07 13:00:33.000000000","message":"Done","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"82046c837dbd9eeec9aa37f34a12ad09b717469f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"               parent_region\u003dNone, **kwargs):"},{"line_number":39,"context_line":"        \"\"\"Create a region."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region."},{"line_number":42,"context_line":"        :param str description: the description the region."},{"line_number":43,"context_line":"        :param bool enabled: whether the endpoint appears in the catalog."},{"line_number":44,"context_line":"        :param parent_region: the parent of the region in the hierarchy."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_a15a1828","line":41,"range":{"start_line":41,"start_character":23,"end_line":41,"end_character":59},"updated":"2016-07-06 09:11:28.000000000","message":"is there a reason we are dropping the details of what happens if the id is pass as none?","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"8d0a03826bb5620f30c9e5b53b985ada0bff378f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"               parent_region\u003dNone, **kwargs):"},{"line_number":39,"context_line":"        \"\"\"Create a region."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region."},{"line_number":42,"context_line":"        :param str description: the description the region."},{"line_number":43,"context_line":"        :param bool enabled: whether the endpoint appears in the catalog."},{"line_number":44,"context_line":"        :param parent_region: the parent of the region in the hierarchy."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_3a0e018c","line":41,"range":{"start_line":41,"start_character":23,"end_line":41,"end_character":59},"in_reply_to":"1aa78d24_3be1289b","updated":"2016-07-07 13:00:33.000000000","message":"Done","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"8d0a03826bb5620f30c9e5b53b985ada0bff378f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"               parent_region\u003dNone, **kwargs):"},{"line_number":39,"context_line":"        \"\"\"Create a region."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region."},{"line_number":42,"context_line":"        :param str description: the description the region."},{"line_number":43,"context_line":"        :param bool enabled: whether the endpoint appears in the catalog."},{"line_number":44,"context_line":"        :param parent_region: the parent of the region in the hierarchy."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_5a0b0d7d","line":41,"range":{"start_line":41,"start_character":23,"end_line":41,"end_character":59},"in_reply_to":"1aa78d24_a15a1828","updated":"2016-07-07 13:00:33.000000000","message":"Done","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"7159858866bc3d25fc21ac2fbabf50f8142860ea","unresolved":false,"context_lines":[{"line_number":38,"context_line":"               parent_region\u003dNone, **kwargs):"},{"line_number":39,"context_line":"        \"\"\"Create a region."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region."},{"line_number":42,"context_line":"        :param str description: the description the region."},{"line_number":43,"context_line":"        :param bool enabled: whether the endpoint appears in the catalog."},{"line_number":44,"context_line":"        :param parent_region: the parent of the region in the hierarchy."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_3be1289b","line":41,"range":{"start_line":41,"start_character":23,"end_line":41,"end_character":59},"in_reply_to":"1aa78d24_a15a1828","updated":"2016-07-06 12:32:03.000000000","message":"no, we should specify the ID will be created by the server","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"dbe6f4c9342620dfd73b83091e78be26d4b63bb7","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        \"\"\"Create a region."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region."},{"line_number":42,"context_line":"        :param str description: the description the region."},{"line_number":43,"context_line":"        :param bool enabled: whether the endpoint appears in the catalog."},{"line_number":44,"context_line":"        :param parent_region: the parent of the region in the hierarchy."},{"line_number":45,"context_line":"        :type parent_region: str or :class:`keystoneclient.v3.regions.Region`"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_8f3e1e8a","line":42,"updated":"2016-07-06 11:12:47.000000000","message":"of the","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"8d0a03826bb5620f30c9e5b53b985ada0bff378f","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        \"\"\"Create a region."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region."},{"line_number":42,"context_line":"        :param str description: the description the region."},{"line_number":43,"context_line":"        :param bool enabled: whether the endpoint appears in the catalog."},{"line_number":44,"context_line":"        :param parent_region: the parent of the region in the hierarchy."},{"line_number":45,"context_line":"        :type parent_region: str or :class:`keystoneclient.v3.regions.Region`"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_5a82edc8","line":42,"in_reply_to":"1aa78d24_8f3e1e8a","updated":"2016-07-07 13:00:33.000000000","message":"Done","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"dbe6f4c9342620dfd73b83091e78be26d4b63bb7","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region."},{"line_number":42,"context_line":"        :param str description: the description the region."},{"line_number":43,"context_line":"        :param bool enabled: whether the endpoint appears in the catalog."},{"line_number":44,"context_line":"        :param parent_region: the parent of the region in the hierarchy."},{"line_number":45,"context_line":"        :type parent_region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":46,"context_line":"        :param kwargs: any other attribute provided will be passed to the"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_6fe65245","line":43,"range":{"start_line":43,"start_character":41,"end_line":43,"end_character":49},"updated":"2016-07-06 11:12:47.000000000","message":"which endpoint? This should be more specific","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"8d0a03826bb5620f30c9e5b53b985ada0bff378f","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region."},{"line_number":42,"context_line":"        :param str description: the description the region."},{"line_number":43,"context_line":"        :param bool enabled: whether the endpoint appears in the catalog."},{"line_number":44,"context_line":"        :param parent_region: the parent of the region in the hierarchy."},{"line_number":45,"context_line":"        :type parent_region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":46,"context_line":"        :param kwargs: any other attribute provided will be passed to the"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_3aea8174","line":43,"range":{"start_line":43,"start_character":41,"end_line":43,"end_character":49},"in_reply_to":"1aa78d24_1bb38c99","updated":"2016-07-07 13:00:33.000000000","message":"Done","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"8d0a03826bb5620f30c9e5b53b985ada0bff378f","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region."},{"line_number":42,"context_line":"        :param str description: the description the region."},{"line_number":43,"context_line":"        :param bool enabled: whether the endpoint appears in the catalog."},{"line_number":44,"context_line":"        :param parent_region: the parent of the region in the hierarchy."},{"line_number":45,"context_line":"        :type parent_region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":46,"context_line":"        :param kwargs: any other attribute provided will be passed to the"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_faf3d970","line":43,"range":{"start_line":43,"start_character":41,"end_line":43,"end_character":49},"in_reply_to":"1aa78d24_6fe65245","updated":"2016-07-07 13:00:33.000000000","message":"Done","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"7159858866bc3d25fc21ac2fbabf50f8142860ea","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region."},{"line_number":42,"context_line":"        :param str description: the description the region."},{"line_number":43,"context_line":"        :param bool enabled: whether the endpoint appears in the catalog."},{"line_number":44,"context_line":"        :param parent_region: the parent of the region in the hierarchy."},{"line_number":45,"context_line":"        :type parent_region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":46,"context_line":"        :param kwargs: any other attribute provided will be passed to the"}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_1bb38c99","line":43,"range":{"start_line":43,"start_character":41,"end_line":43,"end_character":49},"in_reply_to":"1aa78d24_6fe65245","updated":"2016-07-06 12:32:03.000000000","message":"I believe this should be: whether the region is enabled or not, determining if it appears in the catalog.","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"dbe6f4c9342620dfd73b83091e78be26d4b63bb7","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    def get(self, region):"},{"line_number":58,"context_line":"        \"\"\"Retrieve a region."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        :param region: the region to be retrieved from the server."},{"line_number":61,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        :returns: the specified region returned from server."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_6f6bf285","line":60,"updated":"2016-07-06 11:12:47.000000000","message":"would be nice to say that it could be an ID or a region object","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"948ffc179468d7f4b9a94aec8fd259971b60e949","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    def get(self, region):"},{"line_number":58,"context_line":"        \"\"\"Retrieve a region."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        :param region: the region to be retrieved from the server."},{"line_number":61,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        :returns: the specified region returned from server."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_0088f86f","line":60,"in_reply_to":"1aa78d24_1b7a6cbd","updated":"2016-07-07 14:07:27.000000000","message":"Don\u0027t think \"str\" is enough to clarify for the caller that it is an ID, that\u0027s the reason for the suggestion","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"7159858866bc3d25fc21ac2fbabf50f8142860ea","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    def get(self, region):"},{"line_number":58,"context_line":"        \"\"\"Retrieve a region."},{"line_number":59,"context_line":""},{"line_number":60,"context_line":"        :param region: the region to be retrieved from the server."},{"line_number":61,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"        :returns: the specified region returned from server."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_1b7a6cbd","line":60,"in_reply_to":"1aa78d24_6f6bf285","updated":"2016-07-06 12:32:03.000000000","message":"it\u0027s in the \":type region:\" just below","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"dbe6f4c9342620dfd73b83091e78be26d4b63bb7","unresolved":false,"context_lines":[{"line_number":84,"context_line":"               parent_region\u003dNone, **kwargs):"},{"line_number":85,"context_line":"        \"\"\"Update a region."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        :param region: the region to be updated on the server."},{"line_number":88,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":89,"context_line":"        :param str description: the new description of the region."},{"line_number":90,"context_line":"        :param bool enabled: whether the region is enabled."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_af8afa18","line":87,"updated":"2016-07-06 11:12:47.000000000","message":"same comment as above, would be nice to cite that it could be the region_id or a region object","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"dbe6f4c9342620dfd73b83091e78be26d4b63bb7","unresolved":false,"context_lines":[{"line_number":106,"context_line":"    def delete(self, region):"},{"line_number":107,"context_line":"        \"\"\"Delete a region."},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        :param region: the region to be deleted on the server."},{"line_number":110,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        :returns: Request object with 204 status and None as data."}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_4f72764a","line":109,"updated":"2016-07-06 11:12:47.000000000","message":"same as above","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"dbe6f4c9342620dfd73b83091e78be26d4b63bb7","unresolved":false,"context_lines":[{"line_number":109,"context_line":"        :param region: the region to be deleted on the server."},{"line_number":110,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        :returns: Request object with 204 status and None as data."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        \"\"\""},{"line_number":115,"context_line":"        return super(RegionManager, self).delete("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_cf7f8664","line":112,"range":{"start_line":112,"start_character":18,"end_line":112,"end_character":25},"updated":"2016-07-06 11:12:47.000000000","message":"would be nice to point to the object class like it is done above.","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"8d0a03826bb5620f30c9e5b53b985ada0bff378f","unresolved":false,"context_lines":[{"line_number":109,"context_line":"        :param region: the region to be deleted on the server."},{"line_number":110,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        :returns: Request object with 204 status and None as data."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        \"\"\""},{"line_number":115,"context_line":"        return super(RegionManager, self).delete("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_1d9e9fb8","line":112,"range":{"start_line":112,"start_character":18,"end_line":112,"end_character":25},"in_reply_to":"1aa78d24_7b38d0ef","updated":"2016-07-07 13:00:33.000000000","message":"Done","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"7159858866bc3d25fc21ac2fbabf50f8142860ea","unresolved":false,"context_lines":[{"line_number":109,"context_line":"        :param region: the region to be deleted on the server."},{"line_number":110,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        :returns: Request object with 204 status and None as data."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        \"\"\""},{"line_number":115,"context_line":"        return super(RegionManager, self).delete("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_7b38d0ef","line":112,"range":{"start_line":112,"start_character":18,"end_line":112,"end_character":25},"in_reply_to":"1aa78d24_cf7f8664","updated":"2016-07-06 12:32:03.000000000","message":"++ and it\u0027s not a Request object, it\u0027s a Response one","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"8d0a03826bb5620f30c9e5b53b985ada0bff378f","unresolved":false,"context_lines":[{"line_number":109,"context_line":"        :param region: the region to be deleted on the server."},{"line_number":110,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"        :returns: Request object with 204 status and None as data."},{"line_number":113,"context_line":""},{"line_number":114,"context_line":"        \"\"\""},{"line_number":115,"context_line":"        return super(RegionManager, self).delete("}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_dd61b7d4","line":112,"range":{"start_line":112,"start_character":18,"end_line":112,"end_character":25},"in_reply_to":"1aa78d24_cf7f8664","updated":"2016-07-07 13:00:33.000000000","message":"Done","commit_id":"4a728512ec65a10e05854a85ca690f41a33fc4da"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"97c7378bbb1aa1aee66ea125e91bae2a751d2033","unresolved":false,"context_lines":[{"line_number":38,"context_line":"               parent_region\u003dNone, **kwargs):"},{"line_number":39,"context_line":"        \"\"\"Create a region."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region. If not specified a"},{"line_number":42,"context_line":"                       unique identifier will be passed from the server."},{"line_number":43,"context_line":"        :param str description: the description of the region."},{"line_number":44,"context_line":"        :param bool enabled: whether the region is enabled or not, determining"},{"line_number":45,"context_line":"                             if it appears in the catalog."}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_0bc2d9e8","line":42,"range":{"start_line":41,"start_character":77,"end_line":42,"end_character":71},"updated":"2016-07-07 13:42:57.000000000","message":"maybe \u0027an ID will be created by the server\u0027 ?\n\n\u0027passed from the server\u0027 seems weird to me","commit_id":"ca1267c8f700ba954abddb4aa86e5b96583a693d"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"136c2fbc97505ac592df9b9370b20fe01587e57b","unresolved":false,"context_lines":[{"line_number":38,"context_line":"               parent_region\u003dNone, **kwargs):"},{"line_number":39,"context_line":"        \"\"\"Create a region."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region. If not specified a"},{"line_number":42,"context_line":"                       unique identifier will be passed from the server."},{"line_number":43,"context_line":"        :param str description: the description of the region."},{"line_number":44,"context_line":"        :param bool enabled: whether the region is enabled or not, determining"},{"line_number":45,"context_line":"                             if it appears in the catalog."}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_6c25735a","line":42,"range":{"start_line":41,"start_character":77,"end_line":42,"end_character":71},"in_reply_to":"1aa78d24_0bc2d9e8","updated":"2016-07-07 14:12:28.000000000","message":"+1","commit_id":"ca1267c8f700ba954abddb4aa86e5b96583a693d"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"f27dccf106163f8a97e8221e046d08b80dd739e2","unresolved":false,"context_lines":[{"line_number":38,"context_line":"               parent_region\u003dNone, **kwargs):"},{"line_number":39,"context_line":"        \"\"\"Create a region."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"        :param str id: the unique identifier of the region. If not specified a"},{"line_number":42,"context_line":"                       unique identifier will be passed from the server."},{"line_number":43,"context_line":"        :param str description: the description of the region."},{"line_number":44,"context_line":"        :param bool enabled: whether the region is enabled or not, determining"},{"line_number":45,"context_line":"                             if it appears in the catalog."}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_f6392da5","line":42,"range":{"start_line":41,"start_character":77,"end_line":42,"end_character":71},"in_reply_to":"1aa78d24_0bc2d9e8","updated":"2016-07-07 14:58:21.000000000","message":"Done","commit_id":"ca1267c8f700ba954abddb4aa86e5b96583a693d"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"136c2fbc97505ac592df9b9370b20fe01587e57b","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        :param region: the region to be updated on the server."},{"line_number":89,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":90,"context_line":"        :param str description: the new description of the region."},{"line_number":91,"context_line":"        :param bool enabled: whether the region is enabled."},{"line_number":92,"context_line":"        :param parent_region: the new parent of the region in the hierarchy."},{"line_number":93,"context_line":"        :type parent_region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":94,"context_line":"        :param kwargs: any other attribute provided will be passed to server."}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_ac059b46","line":91,"range":{"start_line":91,"start_character":29,"end_line":91,"end_character":58},"updated":"2016-07-07 14:12:28.000000000","message":"this is a update, right? This sentence sounds more like a create action.","commit_id":"ca1267c8f700ba954abddb4aa86e5b96583a693d"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"f27dccf106163f8a97e8221e046d08b80dd739e2","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        :param region: the region to be updated on the server."},{"line_number":89,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":90,"context_line":"        :param str description: the new description of the region."},{"line_number":91,"context_line":"        :param bool enabled: whether the region is enabled."},{"line_number":92,"context_line":"        :param parent_region: the new parent of the region in the hierarchy."},{"line_number":93,"context_line":"        :type parent_region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":94,"context_line":"        :param kwargs: any other attribute provided will be passed to server."}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_396d66cc","line":91,"range":{"start_line":91,"start_character":29,"end_line":91,"end_character":58},"in_reply_to":"1aa78d24_ac059b46","updated":"2016-07-07 14:58:21.000000000","message":"I changed it slightly, please le\u0027me know if you have any other suggestion on this.","commit_id":"ca1267c8f700ba954abddb4aa86e5b96583a693d"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"97c7378bbb1aa1aee66ea125e91bae2a751d2033","unresolved":false,"context_lines":[{"line_number":110,"context_line":"        :param region: the region to be deleted on the server."},{"line_number":111,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        :returns: Response object with 204 status and None as data."},{"line_number":114,"context_line":"        :rtype: :class:`requests.models.Response`"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_e66184d4","line":113,"range":{"start_line":113,"start_character":49,"end_line":113,"end_character":66},"updated":"2016-07-07 13:42:57.000000000","message":"maybe this is not necessary, what do you think ? (I know I had suggested this before)","commit_id":"ca1267c8f700ba954abddb4aa86e5b96583a693d"},{"author":{"_account_id":11022,"name":"Rodrigo Duarte Sousa","email":"rodrigodsousa@gmail.com","username":"rodrigods"},"change_message_id":"136c2fbc97505ac592df9b9370b20fe01587e57b","unresolved":false,"context_lines":[{"line_number":110,"context_line":"        :param region: the region to be deleted on the server."},{"line_number":111,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        :returns: Response object with 204 status and None as data."},{"line_number":114,"context_line":"        :rtype: :class:`requests.models.Response`"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_ccec67bd","line":113,"range":{"start_line":113,"start_character":49,"end_line":113,"end_character":66},"in_reply_to":"1aa78d24_e66184d4","updated":"2016-07-07 14:12:28.000000000","message":"++ looks good either way for me","commit_id":"ca1267c8f700ba954abddb4aa86e5b96583a693d"},{"author":{"_account_id":18176,"name":"Nisha Yadav","email":"ynisha11@gmail.com","username":"ynisha11"},"change_message_id":"f27dccf106163f8a97e8221e046d08b80dd739e2","unresolved":false,"context_lines":[{"line_number":110,"context_line":"        :param region: the region to be deleted on the server."},{"line_number":111,"context_line":"        :type region: str or :class:`keystoneclient.v3.regions.Region`"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        :returns: Response object with 204 status and None as data."},{"line_number":114,"context_line":"        :rtype: :class:`requests.models.Response`"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"1aa78d24_f6912d48","line":113,"range":{"start_line":113,"start_character":49,"end_line":113,"end_character":66},"in_reply_to":"1aa78d24_e66184d4","updated":"2016-07-07 14:58:21.000000000","message":"no problem. Done","commit_id":"ca1267c8f700ba954abddb4aa86e5b96583a693d"}]}