)]}'
{"cliff/tests/test_formatters_table.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"efd42560296805ff020923639a83fbe394b4eb49","unresolved":false,"context_lines":[{"line_number":99,"context_line":"    d \u003d (\u0027A\u0027, \u0027B\u0027, \u0027C\u0027, \u0027d\u0027 * 77)"},{"line_number":100,"context_line":"    assert (expected_ml_val \u003d\u003d"},{"line_number":101,"context_line":"            _table_tester_helper(c, d, extra_args\u003d[\u0027--max-width\u0027, \u002742\u0027]))"},{"line_number":102,"context_line":"    tw.return_value \u003d 0"},{"line_number":103,"context_line":"    assert expected_sl_val \u003d\u003d _table_tester_helper(c, d, extra_args\u003dargs())"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_39d1ac30","line":102,"updated":"2016-06-15 19:29:13.000000000","message":"It was not clear to my why the line 92 was doing 71 * char if the max width was 42 until I realized what this line was doing. This should really be its own test case with a good name to make that clear. Otherwise the implementation looks good.","commit_id":"7cb323616715f4847131b099956a62abada5970a"},{"author":{"_account_id":16318,"name":"Vincent Legoll","email":"vincent.legoll@iphc.cnrs.fr","username":"vincentleg"},"change_message_id":"2de1cec17f742fb78621da35cfe89b00fc7b4fa2","unresolved":false,"context_lines":[{"line_number":99,"context_line":"    d \u003d (\u0027A\u0027, \u0027B\u0027, \u0027C\u0027, \u0027d\u0027 * 77)"},{"line_number":100,"context_line":"    assert (expected_ml_val \u003d\u003d"},{"line_number":101,"context_line":"            _table_tester_helper(c, d, extra_args\u003d[\u0027--max-width\u0027, \u002742\u0027]))"},{"line_number":102,"context_line":"    tw.return_value \u003d 0"},{"line_number":103,"context_line":"    assert expected_sl_val \u003d\u003d _table_tester_helper(c, d, extra_args\u003dargs())"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"7aa08908_a6f1de97","line":102,"in_reply_to":"7aa08908_39d1ac30","updated":"2016-06-16 08:59:27.000000000","message":"I did that way to dodge the pep8 line-too-long error... Yes it\u0027s crude, bordering on the \u0027ugly\u0027 side... I just wanted to get the discussion going about this test, and maybe get someone to give me a better idea.\n\nIs there a way to disable that specific error just for that variable assignment ? That would enable to do it the straight way.\n\nThe comment above \u0027expected_sl_val\u0027 may not be placed at the best position to better explain that...","commit_id":"7cb323616715f4847131b099956a62abada5970a"},{"author":{"_account_id":16318,"name":"Vincent Legoll","email":"vincent.legoll@iphc.cnrs.fr","username":"vincentleg"},"change_message_id":"d9a7734a541bed6ff3d2ec85f52a62c93e81f152","unresolved":false,"context_lines":[{"line_number":99,"context_line":"    d \u003d (\u0027A\u0027, \u0027B\u0027, \u0027C\u0027, \u0027d\u0027 * 77)"},{"line_number":100,"context_line":"    assert (expected_ml_val \u003d\u003d"},{"line_number":101,"context_line":"            _table_tester_helper(c, d, extra_args\u003d[\u0027--max-width\u0027, \u002742\u0027]))"},{"line_number":102,"context_line":"    tw.return_value \u003d 0"},{"line_number":103,"context_line":"    assert expected_sl_val \u003d\u003d _table_tester_helper(c, d, extra_args\u003dargs())"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_62f254d4","line":102,"in_reply_to":"7aa08908_a6f1de97","updated":"2016-06-22 10:26:26.000000000","message":"I\u0027ve been searching for a cleaner solution to this, I found out that pep8, pycodestyle, give differing results for noqa\u0027ed multiline strings... would it be OK to require a specific version of pep8 in test-requirements, to allow usage of \"noqa\" for this ?\n\nBTW, I additionally cut that test into smaller ones (and added others ones...)","commit_id":"7cb323616715f4847131b099956a62abada5970a"}]}