)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"e0f805d6f4a0b720a518b0b5940955eb17de12d7","unresolved":true,"context_lines":[{"line_number":14,"context_line":"The only significant change required here is that we no longer set the"},{"line_number":15,"context_line":"\u0027min_width\u0027 attribute since that actually does something - the wrong"},{"line_number":16,"context_line":"thing - now. We want this attribute to set a lower bound on the wrap"},{"line_number":17,"context_line":"width as opposed to an absolute minimum we can use."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"While we\u0027re here, we also remove a now useless bit of Python 2 code."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"d7eedab8_17b633ae","line":17,"updated":"2021-02-10 18:15:03.000000000","message":"It looks like the min_width value is no longer used at all?","commit_id":"8b937b12b149a7a530bd93a2bb031747fb4acfec"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"4b24af5f2daa4da9314f012e622b235341a69a3e","unresolved":true,"context_lines":[{"line_number":14,"context_line":"The only significant change required here is that we no longer set the"},{"line_number":15,"context_line":"\u0027min_width\u0027 attribute since that actually does something - the wrong"},{"line_number":16,"context_line":"thing - now. We want this attribute to set a lower bound on the wrap"},{"line_number":17,"context_line":"width as opposed to an absolute minimum we can use."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"While we\u0027re here, we also remove a now useless bit of Python 2 code."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"daf38604_1eb2d29c","line":17,"in_reply_to":"d7eedab8_17b633ae","updated":"2021-02-10 19:38:24.000000000","message":"I realize this isn\u0027t clear but I couldn\u0027t find a better way to word it. We still use it here [1]. If we don\u0027t do that, we end up shrinking columns to the width of their title, which for columns like \u0027ID\u0027 is _pretty_ small 😄\n\n[1] https://review.opendev.org/c/openstack/cliff/+/774915/2/cliff/formatters/table.py#204","commit_id":"8b937b12b149a7a530bd93a2bb031747fb4acfec"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"1e6604fd4b9b113e82e3474abda547a079b1b59f","unresolved":false,"context_lines":[{"line_number":14,"context_line":"The only significant change required here is that we no longer set the"},{"line_number":15,"context_line":"\u0027min_width\u0027 attribute since that actually does something - the wrong"},{"line_number":16,"context_line":"thing - now. We want this attribute to set a lower bound on the wrap"},{"line_number":17,"context_line":"width as opposed to an absolute minimum we can use."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"While we\u0027re here, we also remove a now useless bit of Python 2 code."},{"line_number":20,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"48fb1ace_25701c4e","line":17,"in_reply_to":"daf38604_1eb2d29c","updated":"2021-02-10 20:00:52.000000000","message":"OK, I missed that use. Thanks for clarifying.","commit_id":"8b937b12b149a7a530bd93a2bb031747fb4acfec"}]}
