)]}'
{".coveragerc":[{"author":{"_account_id":19220,"name":"Michael Lekkas","email":"mikelekkas@gmail.com","username":"lekkasmi"},"change_message_id":"dc6c774454fcd65ad7449dcaac395d2cfa2c7b99","unresolved":false,"context_lines":[{"line_number":3,"context_line":"source \u003d zun"},{"line_number":4,"context_line":"omit \u003d zun/tests/*"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[report]"},{"line_number":7,"context_line":"exclude_lines \u003d"},{"line_number":8,"context_line":"    pass"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa7ab95a_11236f06","line":6,"updated":"2016-08-28 20:37:21.000000000","message":"Magnum\u0027s .coveragerc has \"ignore_errors \u003d True\".\n\nI have added a comment about that asking clarification in https://review.openstack.org/#/c/361764/1/.coveragerc\n\nDo we want ignore_errors here?","commit_id":"b19cb874268fe4df99434c7e94fead1517effb25"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"c2c4781c910dbd84911eea5eb97cbdef5c8feada","unresolved":false,"context_lines":[{"line_number":3,"context_line":"source \u003d zun"},{"line_number":4,"context_line":"omit \u003d zun/tests/*"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"[report]"},{"line_number":7,"context_line":"exclude_lines \u003d"},{"line_number":8,"context_line":"    pass"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"fa7ab95a_e7eb7f76","line":6,"in_reply_to":"fa7ab95a_11236f06","updated":"2016-08-29 15:55:18.000000000","message":"It looks all other projects put \"ignore_erros \u003d True\" here. For consistency point of view, it is better to set ignore_errors here. Do you have any concern for the ignore_errors?","commit_id":"b19cb874268fe4df99434c7e94fead1517effb25"}]}
