)]}'
{"keystone/common/router.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"af634097ce1c752c44c6274495b1995a7d2b8e67","unresolved":false,"context_lines":[{"line_number":69,"context_line":"             {"},{"line_number":70,"context_line":"                 \u0027href\u0027: collection_path,"},{"line_number":71,"context_line":"             }"},{"line_number":72,"context_line":"             ))"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"        if self._is_entity_implemented:"},{"line_number":75,"context_line":"            entity_rel \u003d json_home.build_v3_resource_relation(self.key)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_7bf4bc5a","line":72,"updated":"2014-08-29 01:10:55.000000000","message":"There\u0027s two indentation styles in this one logical line of code :(","commit_id":"c21368f49e11566d191236ddd9417ba27f6de15d"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"ddab8266c32d7eac15f1aed5db7eccdebe71781e","unresolved":false,"context_lines":[{"line_number":69,"context_line":"             {"},{"line_number":70,"context_line":"                 \u0027href\u0027: collection_path,"},{"line_number":71,"context_line":"             }"},{"line_number":72,"context_line":"             ))"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"        if self._is_entity_implemented:"},{"line_number":75,"context_line":"            entity_rel \u003d json_home.build_v3_resource_relation(self.key)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_317b45bf","line":72,"in_reply_to":"1abeadc6_7bf4bc5a","updated":"2014-08-29 01:56:51.000000000","message":"Done","commit_id":"c21368f49e11566d191236ddd9417ba27f6de15d"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"58b2b8859ec808e85d13ccfe5370d8c3aebb840f","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"        collection_rel \u003d json_home.build_v3_resource_relation("},{"line_number":66,"context_line":"            self.collection_key)"},{"line_number":67,"context_line":"        rel_data \u003d {\u0027href\u0027: collection_path, }"},{"line_number":68,"context_line":"        self._resource_descriptions.append((collection_rel, rel_data))"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"        if self._is_entity_implemented:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_e75abc4e","line":67,"updated":"2014-08-30 03:13:46.000000000","message":"Nice, that is much easier to read!","commit_id":"3a71d9510b8123d87f5881917a01bb830530f517"}],"keystone/common/wsgi.py":[{"author":{"_account_id":5164,"name":"jsimms","email":"justin.simms@gmail.com","username":"jsimms"},"change_message_id":"525de6e0c1089e59eb8830adc501f9f2a7d38112","unresolved":false,"context_lines":[{"line_number":636,"context_line":"                      delete_action\u003dNone, get_post_action\u003dNone, rel\u003dNone,"},{"line_number":637,"context_line":"                      path_vars\u003dNone):"},{"line_number":638,"context_line":""},{"line_number":639,"context_line":"        if not rel:"},{"line_number":640,"context_line":"            raise Exception(\u0027rel parameter is required\u0027)"},{"line_number":641,"context_line":""},{"line_number":642,"context_line":"        if get_head_action:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_5b0620cf","line":639,"updated":"2014-08-29 01:03:29.000000000","message":"if rel is a required parameter, why has it been listed next to last and defaulted? why not move it into the position behind path and take away the default value?","commit_id":"c21368f49e11566d191236ddd9417ba27f6de15d"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"ddab8266c32d7eac15f1aed5db7eccdebe71781e","unresolved":false,"context_lines":[{"line_number":636,"context_line":"                      delete_action\u003dNone, get_post_action\u003dNone, rel\u003dNone,"},{"line_number":637,"context_line":"                      path_vars\u003dNone):"},{"line_number":638,"context_line":""},{"line_number":639,"context_line":"        if not rel:"},{"line_number":640,"context_line":"            raise Exception(\u0027rel parameter is required\u0027)"},{"line_number":641,"context_line":""},{"line_number":642,"context_line":"        if get_head_action:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_91417965","line":639,"in_reply_to":"1abeadc6_5b0620cf","updated":"2014-08-29 01:56:51.000000000","message":"Done","commit_id":"c21368f49e11566d191236ddd9417ba27f6de15d"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"58b2b8859ec808e85d13ccfe5370d8c3aebb840f","unresolved":false,"context_lines":[{"line_number":630,"context_line":"        Use self._add_resource() to map routes for a resource."},{"line_number":631,"context_line":"        \"\"\""},{"line_number":632,"context_line":""},{"line_number":633,"context_line":"    def _add_resource(self, mapper, controller, path, rel,"},{"line_number":634,"context_line":"                      get_action\u003dNone, head_action\u003dNone, get_head_action\u003dNone,"},{"line_number":635,"context_line":"                      put_action\u003dNone, post_action\u003dNone, patch_action\u003dNone,"},{"line_number":636,"context_line":"                      delete_action\u003dNone, get_post_action\u003dNone,"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_47337013","line":633,"updated":"2014-08-30 03:13:46.000000000","message":"That is an amazingly simple way to make something required! I\u0027m gonna have to remember that trick :-)","commit_id":"3a71d9510b8123d87f5881917a01bb830530f517"}]}