)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"623b86374d6804598ec698370cc193588eea3951","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix flake8 E12[3-8] for ironic/tests/drivers"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is a section of a larger effort to fix and enforce the rules"},{"line_number":10,"context_line":"E123 to E128 in the ironic code base."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ia96582b5e9abc088d6c1694afc93c59be4a4065c"},{"line_number":13,"context_line":"Partial-Bug: 1421522"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5afe65bd_481d9f0e","line":10,"updated":"2015-06-01 16:02:54.000000000","message":"I would like more explanation here. I think the code already conforms to some E12* rules. Or have we violated all E123, 124, 125, 126, 127, 128? And it would help to put a short description of what those E12* are that you are fixing.","commit_id":"7ca7b182d8836a53263964ecdd61c102b869898e"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"79268b1c1f57ccadee6f89d62e88c407d88c1495","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix flake8 E12[3-8] for ironic/tests/drivers"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is a section of a larger effort to fix and enforce the rules"},{"line_number":10,"context_line":"E123 to E128 in the ironic code base."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ia96582b5e9abc088d6c1694afc93c59be4a4065c"},{"line_number":13,"context_line":"Partial-Bug: 1421522"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5afe65bd_c934d5df","line":10,"in_reply_to":"5afe65bd_0e1a0f6f","updated":"2015-06-02 09:46:46.000000000","message":"Done","commit_id":"7ca7b182d8836a53263964ecdd61c102b869898e"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"0c7999d904a2f52d6418efe0a03f582390d437c0","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix flake8 E12[3-8] for ironic/tests/drivers"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is a section of a larger effort to fix and enforce the rules"},{"line_number":10,"context_line":"E123 to E128 in the ironic code base."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ia96582b5e9abc088d6c1694afc93c59be4a4065c"},{"line_number":13,"context_line":"Partial-Bug: 1421522"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5afe65bd_8b3f11d6","line":10,"in_reply_to":"5afe65bd_481d9f0e","updated":"2015-06-01 16:35:00.000000000","message":"+1","commit_id":"7ca7b182d8836a53263964ecdd61c102b869898e"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"b834fe0911643e9c548bc21e12379a2d5705b96a","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Fix flake8 E12[3-8] for ironic/tests/drivers"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This is a section of a larger effort to fix and enforce the rules"},{"line_number":10,"context_line":"E123 to E128 in the ironic code base."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ia96582b5e9abc088d6c1694afc93c59be4a4065c"},{"line_number":13,"context_line":"Partial-Bug: 1421522"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"5afe65bd_0e1a0f6f","line":10,"in_reply_to":"5afe65bd_8b3f11d6","updated":"2015-06-01 16:37:07.000000000","message":"I think it is only doing:\n E123 (*) \tclosing bracket does not match indentation of opening bracket’s line\n E126 (*^) \tcontinuation line over-indented for hanging indent\n E127 (^) \tcontinuation line over-indented for visual indent\n E128 (^) \tcontinuation line under-indented for visual indent","commit_id":"7ca7b182d8836a53263964ecdd61c102b869898e"}],"ironic/tests/drivers/drac/test_management.py":[{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"5a2483cb28cf2fdb104e240737b0634c7e8d2437","unresolved":false,"context_lines":[{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    def test__get_next_boot_mode(self, mock_client_pywsman):"},{"line_number":52,"context_line":"        result_xml \u003d test_utils.build_soap_xml("},{"line_number":53,"context_line":"            [{\u0027DCIM_BootConfigSetting\u0027:"},{"line_number":54,"context_line":"                {\u0027InstanceID\u0027: \u0027IPL\u0027,"},{"line_number":55,"context_line":"                 \u0027IsNext\u0027: drac_mgmt.PERSISTENT}}],"},{"line_number":56,"context_line":"            resource_uris.DCIM_BootConfigSetting)"}],"source_content_type":"text/x-python","patch_set":5,"id":"5afe65bd_648fe42a","line":53,"updated":"2015-06-04 10:03:29.000000000","message":"May be it\u0027s just my preference, but do you feel it\u0027s better if it\u0027s like:\nhttp://paste.openstack.org/show/262360/\n\nDitto question for below ones also.","commit_id":"a018e938ad79cf196c8a654cfea66b9612e3825d"}],"ironic/tests/drivers/ilo/test_common.py":[{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"623b86374d6804598ec698370cc193588eea3951","unresolved":false,"context_lines":[{"line_number":430,"context_line":"                                  shared\u003dFalse) as task:"},{"line_number":431,"context_line":"            ilo_common.setup_vmedia_for_boot(task, boot_iso)"},{"line_number":432,"context_line":"            swift_obj_mock.get_temp_url.assert_called_once_with("},{"line_number":433,"context_line":"                \u0027ilo_cont\u0027, \u0027object-name\u0027, 1)"},{"line_number":434,"context_line":"            attach_vmedia_mock.assert_called_once_with("},{"line_number":435,"context_line":"                task.node, \u0027CDROM\u0027, \u0027image_url\u0027)"},{"line_number":436,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5afe65bd_a85773c9","line":433,"updated":"2015-06-01 16:02:54.000000000","message":"this was originally indented 8 chars. What\u0027s the hacking rule that causes it to be indented 4 now?","commit_id":"7ca7b182d8836a53263964ecdd61c102b869898e"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"0c7999d904a2f52d6418efe0a03f582390d437c0","unresolved":false,"context_lines":[{"line_number":430,"context_line":"                                  shared\u003dFalse) as task:"},{"line_number":431,"context_line":"            ilo_common.setup_vmedia_for_boot(task, boot_iso)"},{"line_number":432,"context_line":"            swift_obj_mock.get_temp_url.assert_called_once_with("},{"line_number":433,"context_line":"                \u0027ilo_cont\u0027, \u0027object-name\u0027, 1)"},{"line_number":434,"context_line":"            attach_vmedia_mock.assert_called_once_with("},{"line_number":435,"context_line":"                task.node, \u0027CDROM\u0027, \u0027image_url\u0027)"},{"line_number":436,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"5afe65bd_2baa5d1a","line":433,"in_reply_to":"5afe65bd_a85773c9","updated":"2015-06-01 16:35:00.000000000","message":"I would think it would be:\n\n E127 (^) \tcontinuation line over-indented for visual indent","commit_id":"7ca7b182d8836a53263964ecdd61c102b869898e"}],"ironic/tests/drivers/ilo/test_common.py.orig":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"12f9a9dc1391cd796b7db328d16f57b06f6cd260","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"5afe65bd_cb65e0e1","updated":"2015-06-02 14:25:57.000000000","message":"Ooops, this file should not be here :)","commit_id":"080b1e23be6734404ebb5e09930d71f4feeeb030"}]}