)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"95acf4e4a67af6d499f78f53b35f8ab494c281e2","unresolved":false,"context_lines":[{"line_number":11,"context_line":"test codebase. To have single point of modification it\u0027s better to have"},{"line_number":12,"context_line":"common class for controllers tests."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This change replaces webop.Request to fakes.HTTPRequest and updates"},{"line_number":15,"context_line":"blank method signature with respect to actual webop.Request.blank"},{"line_number":16,"context_line":"method."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9ad45d7e_6b12d90a","line":14,"updated":"2016-08-10 11:09:17.000000000","message":"s/webop/webob/\n\n(I do that all the time too)","commit_id":"c7ac03108fe461c01f654967e03a84e1f0fdade1"},{"author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"change_message_id":"695a5712d2bf5b7c034dc0631bbea1b9e3377e0d","unresolved":false,"context_lines":[{"line_number":11,"context_line":"test codebase. To have single point of modification it\u0027s better to have"},{"line_number":12,"context_line":"common class for controllers tests."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This change replaces webop.Request to fakes.HTTPRequest and updates"},{"line_number":15,"context_line":"blank method signature with respect to actual webop.Request.blank"},{"line_number":16,"context_line":"method."},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9ad45d7e_b12440c5","line":14,"in_reply_to":"9ad45d7e_6b12d90a","updated":"2016-08-10 11:45:39.000000000","message":"Done","commit_id":"c7ac03108fe461c01f654967e03a84e1f0fdade1"},{"author":{"_account_id":14819,"name":"Timofey Durakov","email":"timofei.nd@gmail.com","username":"tdurakov"},"change_message_id":"c99d3e64f63fdaaa10f1e021fa45b9b95c37922c","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Tests: use fakes.HTTPRequest in compute tests"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In wsgi controllers tests fakes.HTTPRequest and webob.Request are used."},{"line_number":10,"context_line":"fakes. HTTPRequest is the descendant of webob.Request defined in nova"},{"line_number":11,"context_line":"test codebase. To have single point of modification it\u0027s better to have"},{"line_number":12,"context_line":"common class for controllers tests."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"3ac371cc_020449c4","line":10,"range":{"start_line":10,"start_character":6,"end_line":10,"end_character":7},"updated":"2016-08-19 06:41:38.000000000","message":"super-nit: redundant space","commit_id":"b2fd7225ae1fb68927b03c6025656cdb3d79473d"}],"nova/tests/unit/api/openstack/compute/test_api.py":[{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"f04ee8eb737a930525fd8a37042634a9c68fb2d4","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from oslo_serialization import jsonutils"},{"line_number":17,"context_line":"import six"},{"line_number":18,"context_line":"import webob"},{"line_number":19,"context_line":"import webob.dec"},{"line_number":20,"context_line":"import webob.exc"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3ac371cc_bd5e0810","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":12},"updated":"2016-08-16 07:25:07.000000000","message":"Is this line necessary? I think it is for only \"webob.Request\". It looks enough \"webob.dec\" and \"webob.exc\".","commit_id":"0ec1e46213da4954be95e526775a53f93312f904"},{"author":{"_account_id":17255,"name":"Toshiaki Takahashi","email":"ts-takahashi@nec.com","username":"takahashi_ts"},"change_message_id":"07d1bf407a121dd6cb4e85f9f8006120b2deb44a","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from oslo_serialization import jsonutils"},{"line_number":17,"context_line":"import six"},{"line_number":18,"context_line":"import webob"},{"line_number":19,"context_line":"import webob.dec"},{"line_number":20,"context_line":"import webob.exc"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3ac371cc_a302f107","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":12},"in_reply_to":"3ac371cc_6897a690","updated":"2016-08-18 00:52:11.000000000","message":"I mean that it does not require \"import webob\" because there is a line \"import webob.exc\". Is this understanding mistaken?","commit_id":"0ec1e46213da4954be95e526775a53f93312f904"},{"author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"change_message_id":"46520a5f78809885cb0c63bc3a4cac590b9a4fc6","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from oslo_serialization import jsonutils"},{"line_number":17,"context_line":"import six"},{"line_number":18,"context_line":"import webob"},{"line_number":19,"context_line":"import webob.dec"},{"line_number":20,"context_line":"import webob.exc"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3ac371cc_ba231555","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":12},"in_reply_to":"3ac371cc_a302f107","updated":"2016-08-18 11:41:38.000000000","message":"Yes, you are right. Didn\u0027t understand it at the first time, sorry.","commit_id":"0ec1e46213da4954be95e526775a53f93312f904"},{"author":{"_account_id":21813,"name":"Andrey Volkov","email":"m@amadev.ru","username":"avolkov"},"change_message_id":"2553843b4b9f2648c1d17c5c76f044bcad307bec","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from oslo_serialization import jsonutils"},{"line_number":17,"context_line":"import six"},{"line_number":18,"context_line":"import webob"},{"line_number":19,"context_line":"import webob.dec"},{"line_number":20,"context_line":"import webob.exc"},{"line_number":21,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3ac371cc_6897a690","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":12},"in_reply_to":"3ac371cc_bd5e0810","updated":"2016-08-16 10:42:32.000000000","message":"It\u0027s actually used on https://github.com/openstack/nova/blob/master/nova/tests/unit/api/openstack/compute/test_api.py#L90","commit_id":"0ec1e46213da4954be95e526775a53f93312f904"}],"nova/tests/unit/api/openstack/fakes.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"f6e5d8dd54db7bdf98f04c5e82d06ac04a8b5b18","unresolved":false,"context_lines":[{"line_number":224,"context_line":""},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"class HTTPRequestV21(HTTPRequest):"},{"line_number":227,"context_line":"    pass"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"class TestRouter(wsgi.Router):"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa7ab95a_7b5f7c72","line":227,"updated":"2016-08-26 12:27:48.000000000","message":"it would be nice to get rid of this class all together.","commit_id":"d995e2eb9614cbe9a4b7d541cfd38da1b9daf4ee"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"9fea1f66e0ebfffbf7c6af6394f8f9b6afce3b3a","unresolved":false,"context_lines":[{"line_number":224,"context_line":""},{"line_number":225,"context_line":""},{"line_number":226,"context_line":"class HTTPRequestV21(HTTPRequest):"},{"line_number":227,"context_line":"    pass"},{"line_number":228,"context_line":""},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"class TestRouter(wsgi.Router):"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa7ab95a_987f3f14","line":227,"in_reply_to":"fa7ab95a_7b5f7c72","updated":"2016-08-30 11:08:50.000000000","message":"+1","commit_id":"d995e2eb9614cbe9a4b7d541cfd38da1b9daf4ee"}]}