)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"56282b284fd5382fba373d57b87db02ef8f83b25","unresolved":false,"context_lines":[{"line_number":17,"context_line":"magnumclient/tests/test_utils.py"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Still need the encoding for Non-ASCII characters in the file"},{"line_number":20,"context_line":"magnumclient/common/httpclient.py"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I07941effe5803572e3a8b1c5c0a86b4c7fd259ff"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"7a3c09a3_5f6574df","line":20,"updated":"2017-01-13 04:22:47.000000000","message":"Can you tell in which line the non-ASCII character is there","commit_id":"e4b0a740daf923def8fcaba53816425f1810fa40"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"1ac8ec3f694e6de5b36ab0d3ea9f6aa00814238f","unresolved":false,"context_lines":[{"line_number":17,"context_line":"magnumclient/tests/test_utils.py"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Still need the encoding for Non-ASCII characters in the file"},{"line_number":20,"context_line":"magnumclient/common/httpclient.py at line no: 194"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Change-Id: I07941effe5803572e3a8b1c5c0a86b4c7fd259ff"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7a3c09a3_ff09bdde","line":20,"updated":"2017-01-13 06:57:24.000000000","message":"How you judged that utf-8 encoding on beginning of file and on line 194 is related. Any reference?\n\nI think if you use any unicode character in the file then you need to add utf-8 encoding in the beginning of the file.\n\nDo the tests fail if you remove utf-8 encoding line from magnumclient/common/httpclient.py","commit_id":"10e1523286de52e3f1a2adc76659e6f6b0986d4d"}],"tox.ini":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"56282b284fd5382fba373d57b87db02ef8f83b25","unresolved":false,"context_lines":[{"line_number":53,"context_line":"# E123, E125 skipped as they are invalid PEP-8."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"show-source \u003d True"},{"line_number":56,"context_line":"ignore \u003d E123,E125,H903"},{"line_number":57,"context_line":"builtins \u003d _"},{"line_number":58,"context_line":"exclude\u003d.venv,.git,.tox,dist,doc,,*lib/python*,*egg,build"},{"line_number":59,"context_line":""}],"source_content_type":"text/x-properties","patch_set":1,"id":"7a3c09a3_df9fa4b2","line":56,"updated":"2017-01-13 04:22:47.000000000","message":"what the reason to add H903 in the ignore list?","commit_id":"e4b0a740daf923def8fcaba53816425f1810fa40"}]}