)]}'
{"tests/test_build.py":[{"author":{"_account_id":23717,"name":"Surya Prakash Singh","email":"singh.surya64mnnit@gmail.com","username":"SuryaPrakash"},"change_message_id":"f5258402eb7186c6df02938353a5d59bfe41d234","unresolved":false,"context_lines":[{"line_number":140,"context_line":"                                \"--type\", \"source\"])"},{"line_number":141,"context_line":""},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"class BuildTestDebianBinary(BuildTestUbuntuBinary):"},{"line_number":144,"context_line":"    excluded_images \u003d ["},{"line_number":145,"context_line":"        \"bifrost-base\","},{"line_number":146,"context_line":"        \"cloudkitty-base\","}],"source_content_type":"text/x-python","patch_set":6,"id":"ba2be162_8bceaa81","line":143,"range":{"start_line":143,"start_character":28,"end_line":143,"end_character":49},"updated":"2017-03-01 09:03:24.000000000","message":"I think it should be like this (BuildTest, base.BaseTestCase), if we are inheriting the BuildTestDebianBinary class, then we should not again define the *excluded_images* variable with the same value. Which irrelevant. Please correct if I am wrong.","commit_id":"255d8d39aecb897939fb7c8e72c339dce19ae707"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"0ec4805e5308c122814c5a2bbe669524aff7601d","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        \"vitrage-base\","},{"line_number":161,"context_line":"        \"vmtp\","},{"line_number":162,"context_line":"        \"zaqar\","},{"line_number":163,"context_line":"        \"zun-base\""},{"line_number":164,"context_line":"    ]"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a30ddce_61439e13","line":163,"updated":"2017-03-13 05:30:49.000000000","message":"last , is unnecessary in list.","commit_id":"443694bcfa37a1e7a8f16c65ab173647a2b69af2"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"8e6fbd76ad4e166359225793c05f5dbf993cd2af","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        \"vitrage-base\","},{"line_number":161,"context_line":"        \"vmtp\","},{"line_number":162,"context_line":"        \"zaqar\","},{"line_number":163,"context_line":"        \"zun-base\""},{"line_number":164,"context_line":"    ]"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a30ddce_a8403708","line":163,"range":{"start_line":163,"start_character":16,"end_line":163,"end_character":18},"updated":"2017-03-10 16:50:53.000000000","message":"need a  \",\" here","commit_id":"443694bcfa37a1e7a8f16c65ab173647a2b69af2"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"53d41bc7567f7e9cb367940b4f864c6059f34e17","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        \"vitrage-base\","},{"line_number":161,"context_line":"        \"vmtp\","},{"line_number":162,"context_line":"        \"zaqar\","},{"line_number":163,"context_line":"        \"zun-base\""},{"line_number":164,"context_line":"    ]"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a30ddce_c791a848","line":163,"in_reply_to":"9a30ddce_61439e13","updated":"2017-03-13 14:51:04.000000000","message":"please check the line 125 or line 198, if the \",\" is unnecessary on last,  it should keep consistent with others","commit_id":"443694bcfa37a1e7a8f16c65ab173647a2b69af2"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"36a16a8dacf85d83cc7c2e4523724ad6d68e895d","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        \"vitrage-base\","},{"line_number":161,"context_line":"        \"vmtp\","},{"line_number":162,"context_line":"        \"zaqar\","},{"line_number":163,"context_line":"        \"zun-base\""},{"line_number":164,"context_line":"    ]"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"3a1ff146_6f889ee8","line":163,"in_reply_to":"9a30ddce_8780908c","updated":"2017-03-15 10:11:02.000000000","message":"adding , at the end is just to avoid merge more merge conflicts than necessary, but not needed at all.","commit_id":"443694bcfa37a1e7a8f16c65ab173647a2b69af2"},{"author":{"_account_id":23717,"name":"Surya Prakash Singh","email":"singh.surya64mnnit@gmail.com","username":"SuryaPrakash"},"change_message_id":"f4efc1ac2bcc64ddeba3aa4740435a04bf39c04c","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        \"vitrage-base\","},{"line_number":161,"context_line":"        \"vmtp\","},{"line_number":162,"context_line":"        \"zaqar\","},{"line_number":163,"context_line":"        \"zun-base\""},{"line_number":164,"context_line":"    ]"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a30ddce_b4616407","line":163,"range":{"start_line":163,"start_character":16,"end_line":163,"end_character":18},"in_reply_to":"9a30ddce_a8403708","updated":"2017-03-11 01:26:20.000000000","message":"why ?? that is the last item in the list.","commit_id":"443694bcfa37a1e7a8f16c65ab173647a2b69af2"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"256053f5a7c5db0ce91da5b9d77706b38733834f","unresolved":false,"context_lines":[{"line_number":160,"context_line":"        \"vitrage-base\","},{"line_number":161,"context_line":"        \"vmtp\","},{"line_number":162,"context_line":"        \"zaqar\","},{"line_number":163,"context_line":"        \"zun-base\""},{"line_number":164,"context_line":"    ]"},{"line_number":165,"context_line":""},{"line_number":166,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a30ddce_8780908c","line":163,"in_reply_to":"9a30ddce_c791a848","updated":"2017-03-13 14:58:37.000000000","message":"pep8 works ok, the \",\" is unnecessary","commit_id":"443694bcfa37a1e7a8f16c65ab173647a2b69af2"}]}