)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"ef7824fd1f4766a8a63868adf2238222482d113c","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"We\u0027ve discussed using black before and decided it would probably be a"},{"line_number":17,"context_line":"bad idea for a few reasons, chief among them being the breaking of"},{"line_number":18,"context_line":"git-blame, but this particular choice of black has some merit, and I"},{"line_number":19,"context_line":"have noticed I have noticed that it leads to a significantly reduced"},{"line_number":20,"context_line":"amount of unnecessary line wrapping when used in other projects."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"There\u0027s no reason to go wrap *existing* code but for new or reworked"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_dc98661b","line":19,"range":{"start_line":18,"start_character":67,"end_line":19,"end_character":27},"updated":"2020-02-06 16:07:27.000000000","message":"you could significantly reduce the amount of unnecessary line wrapping here.","commit_id":"a531a58bf09db76cc9a3ab6efd320c4755cd7753"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"77d2798258734a869d85eab88e44185f094b8f85","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"We\u0027ve discussed using black before and decided it would probably be a"},{"line_number":17,"context_line":"bad idea for a few reasons, chief among them being the breaking of"},{"line_number":18,"context_line":"git-blame, but this particular choice of black has some merit, and I"},{"line_number":19,"context_line":"have noticed I have noticed that it leads to a significantly reduced"},{"line_number":20,"context_line":"amount of unnecessary line wrapping when used in other projects."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"There\u0027s no reason to go wrap *existing* code but for new or reworked"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_18711c46","line":19,"range":{"start_line":18,"start_character":67,"end_line":19,"end_character":27},"in_reply_to":"3fa7e38b_dc98661b","updated":"2020-02-07 10:24:23.000000000","message":"blah","commit_id":"a531a58bf09db76cc9a3ab6efd320c4755cd7753"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"094114e37ba45e98ddbc49ed5ff4a44d13d34bcd","unresolved":false,"context_lines":[{"line_number":21,"context_line":""},{"line_number":22,"context_line":"There\u0027s no reason to go wrap *existing* code but for new or reworked"},{"line_number":23,"context_line":"code, this should make our lives a little easier. The main disadvantage"},{"line_number":24,"context_line":"of bumping this number would be backports, which would need to be"},{"line_number":25,"context_line":"rewrapped in order to avoid breaking flake8 checks on those branches."},{"line_number":26,"context_line":"However, there\u0027s no reason we can\u0027t backport this fix if we decided it"},{"line_number":27,"context_line":"would be helpful, or simply modify the backport to stick with the older"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_4ee02efa","line":24,"range":{"start_line":24,"start_character":32,"end_line":24,"end_character":41},"updated":"2020-02-07 12:54:41.000000000","message":"Backporting is hard when we do refactorings or code removals. But we do them as we feel them necessary for the health of the codebase. So I can accept a similar argument here.","commit_id":"a531a58bf09db76cc9a3ab6efd320c4755cd7753"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"094114e37ba45e98ddbc49ed5ff4a44d13d34bcd","unresolved":false,"context_lines":[{"line_number":25,"context_line":"rewrapped in order to avoid breaking flake8 checks on those branches."},{"line_number":26,"context_line":"However, there\u0027s no reason we can\u0027t backport this fix if we decided it"},{"line_number":27,"context_line":"would be helpful, or simply modify the backport to stick with the older"},{"line_number":28,"context_line":"rule."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"[1] https://black.readthedocs.io/en/stable/the_black_code_style.html#line-length"},{"line_number":31,"context_line":"[2] https://youtu.be/wf-BqAjZb8M?t\u003d260"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_ae0be2d1","line":28,"updated":"2020-02-07 12:54:41.000000000","message":"What about going insane with black + git hyper-blame ?\n\n... I\u0027ll let myself out.","commit_id":"a531a58bf09db76cc9a3ab6efd320c4755cd7753"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"094114e37ba45e98ddbc49ed5ff4a44d13d34bcd","unresolved":false,"context_lines":[{"line_number":28,"context_line":"rule."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"[1] https://black.readthedocs.io/en/stable/the_black_code_style.html#line-length"},{"line_number":31,"context_line":"[2] https://youtu.be/wf-BqAjZb8M?t\u003d260"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"Change-Id: I65a7cd92c6908ee18495134d48a987d2351ca822"},{"line_number":34,"context_line":"Signed-off-by: Stephen Finucane \u003csfinucan@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_ee5c7af0","line":31,"range":{"start_line":31,"start_character":1,"end_line":31,"end_character":38},"updated":"2020-02-07 12:54:41.000000000","message":"queued up for viewing as it sounds interesting","commit_id":"a531a58bf09db76cc9a3ab6efd320c4755cd7753"}]}