)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"33d57fffa85dba5bbcf9083ef84056a292f558b1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove stubs and add resource cleanup"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds cleanup resource and removes stubs in test_admin_actions.py."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I08df6c26b4c7cb17c12248a8cf1f5ed647e5fa17"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a8ffd7b_aa4650bd","line":9,"updated":"2015-11-24 17:07:25.000000000","message":"We tend to wrap lines at 72 column in commit message description (50 in title).\n\nIf you\u0027ll be forced to do another patchset consider making this more descriptive. Why are you actually removing the stubs? I think these are unused, but that\u0027s not obvious when looking at the code (on the other hand cleanup is obvious :)).","commit_id":"9a7e723fe12315325e61bbefddcc50b76d1c1e08"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"8ea71adfa53c81a5d30d9c68369c122f6c7ac662","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove stubs and add resource cleanup"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds cleanup resource and removes stubs in test_admin_actions.py."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I08df6c26b4c7cb17c12248a8cf1f5ed647e5fa17"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a8ffd7b_cd0c0e45","line":9,"in_reply_to":"9a8ffd7b_aa4650bd","updated":"2015-11-24 17:27:00.000000000","message":"Not clear to me why the stubs are removed either.  Can you explain?","commit_id":"9a7e723fe12315325e61bbefddcc50b76d1c1e08"},{"author":{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},"change_message_id":"af7d41a6abd46d9f64cb68d5d88a306501c0a1d6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove stubs and add resource cleanup"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch adds cleanup resource and removes stubs in test_admin_actions.py."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I08df6c26b4c7cb17c12248a8cf1f5ed647e5fa17"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"9a8ffd7b_4b7321b2","line":9,"in_reply_to":"9a8ffd7b_cd0c0e45","updated":"2015-11-24 19:12:56.000000000","message":"First, stubs isn\u0027t used in this test, otherwise it could be replaced with mock.","commit_id":"9a7e723fe12315325e61bbefddcc50b76d1c1e08"}]}