)]}'
{"neutron/tests/unit/hyperv/test_hyperv_utilsv2.py":[{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"95c84fbec032aed8282b313b1beb2824bc7beae9","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        mock_vm \u003d mock.MagicMock()"},{"line_number":109,"context_line":"        mock_vm.path_.return_value \u003d self._FAKE_VM_PATH"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        self._utils._check_job_status \u003d mock.MagicMock()"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        self._utils._add_virt_feature(mock_vm, mock_res_setting_data)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F8pK8%3D","line":110,"updated":"2013-12-23 20:11:45.000000000","message":"nit: but whats the rhythm or reason to when a blank line is inserted or not? Generally we don\u0027t insert newlines, see http://docs.openstack.org/developer/hacking/","commit_id":"a11232a02f746c3a732a23a1f7e04713412e8c85"},{"author":{"_account_id":5572,"name":"Don Kehn","display_name":"DEKehn","email":"dekehn@gmail.com","username":"dekehn"},"change_message_id":"dae6cb6bdcbffdff9b75a66b67968606881ff25c","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        mock_vm \u003d mock.MagicMock()"},{"line_number":109,"context_line":"        mock_vm.path_.return_value \u003d self._FAKE_VM_PATH"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        self._utils._check_job_status \u003d mock.MagicMock()"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        self._utils._add_virt_feature(mock_vm, mock_res_setting_data)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F8lu4%3D","line":110,"in_reply_to":"AAAATn%2F8pJQ%3D","updated":"2013-12-24 20:45:41.000000000","message":"Understood.","commit_id":"a11232a02f746c3a732a23a1f7e04713412e8c85"},{"author":{"_account_id":3185,"name":"Alessandro Pilotti","email":"apilotti@cloudbasesolutions.com","username":"alexpilotti"},"change_message_id":"eedafdff20056d18810edcc86bd92126dfd78cc8","unresolved":false,"context_lines":[{"line_number":107,"context_line":""},{"line_number":108,"context_line":"        mock_vm \u003d mock.MagicMock()"},{"line_number":109,"context_line":"        mock_vm.path_.return_value \u003d self._FAKE_VM_PATH"},{"line_number":110,"context_line":""},{"line_number":111,"context_line":"        self._utils._check_job_status \u003d mock.MagicMock()"},{"line_number":112,"context_line":""},{"line_number":113,"context_line":"        self._utils._add_virt_feature(mock_vm, mock_res_setting_data)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F8pJQ%3D","line":110,"in_reply_to":"AAAATn%2F8pK8%3D","updated":"2013-12-23 20:21:58.000000000","message":"Just for clarity. It\u0027s not clear to me what the difference between this specific newline and others in the code.\n\nNote: this patch is a backport to Havana. So it should be a simple cherry pick from the corresponding Icehouse patch unless specific merge issues occur..","commit_id":"a11232a02f746c3a732a23a1f7e04713412e8c85"}]}