)]}'
{"tempest/lib/common/rest_client.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"772963b1701c3f2b4aa5b921127f6df4ad225d13","unresolved":false,"context_lines":[{"line_number":842,"context_line":"                #             \u0027properties\u0027: {"},{"line_number":843,"context_line":"                #                 \u0027keypair\u0027: {"},{"line_number":844,"context_line":"                #                      ..."},{"line_number":845,"context_line":"                first_key, first_item \u003d six.next(six.iteritems(schema))"},{"line_number":846,"context_line":"                expected_status \u003d int(first_key)"},{"line_number":847,"context_line":"                body_schema \u003d first_item.get(\u0027schema\u0027)"},{"line_number":848,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_0b3aade7","line":845,"updated":"2016-04-08 08:45:46.000000000","message":"I suggest:\n  key, item \u003d schema.popitem()\n\nThere\u0027s no need to build an iterable if you are only interested in one item. Beside, Python dict are unordored, so there\u0027s no \"first_key\", because there\u0027s no order. Here your code works because your dict as only one top level key.","commit_id":"d788b058a6795626734b514e7d3c3cbb3b656873"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"da2768e0339bbfe7be4b8a8685c806c21c34b4e9","unresolved":false,"context_lines":[{"line_number":842,"context_line":"                #             \u0027properties\u0027: {"},{"line_number":843,"context_line":"                #                 \u0027keypair\u0027: {"},{"line_number":844,"context_line":"                #                      ..."},{"line_number":845,"context_line":"                first_key, first_item \u003d six.next(six.iteritems(schema))"},{"line_number":846,"context_line":"                expected_status \u003d int(first_key)"},{"line_number":847,"context_line":"                body_schema \u003d first_item.get(\u0027schema\u0027)"},{"line_number":848,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_7c8791d7","line":845,"in_reply_to":"9a061dce_0b3aade7","updated":"2016-04-08 16:52:09.000000000","message":"Nice suggestion. Thanks, Jordan","commit_id":"d788b058a6795626734b514e7d3c3cbb3b656873"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"f12cd9a2a30c8cd38a7fb7febb8a9651655853b1","unresolved":false,"context_lines":[{"line_number":842,"context_line":"                #             \u0027properties\u0027: {"},{"line_number":843,"context_line":"                #                 \u0027keypair\u0027: {"},{"line_number":844,"context_line":"                #                      ..."},{"line_number":845,"context_line":"                first_key, first_item \u003d six.next(six.iteritems(schema))"},{"line_number":846,"context_line":"                expected_status \u003d int(first_key)"},{"line_number":847,"context_line":"                body_schema \u003d first_item.get(\u0027schema\u0027)"},{"line_number":848,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_a7e756ab","line":845,"in_reply_to":"9a061dce_7c8791d7","updated":"2016-04-08 17:13:44.000000000","message":"Oops. popitem() is not right here, because the method removes original schema when calling it at the first time.","commit_id":"d788b058a6795626734b514e7d3c3cbb3b656873"}]}