)]}'
{"openstackclient/tests/network/v2/test_network.py":[{"author":{"_account_id":9459,"name":".Haifeng Yan","email":"yanheven@qq.com","username":"yan"},"change_message_id":"98a212118f22988184403366e7dd454dac346f84","unresolved":false,"context_lines":[{"line_number":298,"context_line":"        \u0027ID\u0027,"},{"line_number":299,"context_line":"        \u0027Name\u0027,"},{"line_number":300,"context_line":"        \u0027Subnets\u0027"},{"line_number":301,"context_line":"    ]"},{"line_number":302,"context_line":"    columns_long \u003d ["},{"line_number":303,"context_line":"        \u0027ID\u0027,"},{"line_number":304,"context_line":"        \u0027Name\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_0f58bef9","line":301,"updated":"2015-12-04 07:30:40.000000000","message":"const variable should be upper case. also I suggest let them in the beginning of module.","commit_id":"697df67611e85116e67e65d6c28b938c172e89ef"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"df6c95c5937f5306f2a50b22e2179a3029b9a92e","unresolved":false,"context_lines":[{"line_number":298,"context_line":"        \u0027ID\u0027,"},{"line_number":299,"context_line":"        \u0027Name\u0027,"},{"line_number":300,"context_line":"        \u0027Subnets\u0027"},{"line_number":301,"context_line":"    ]"},{"line_number":302,"context_line":"    columns_long \u003d ["},{"line_number":303,"context_line":"        \u0027ID\u0027,"},{"line_number":304,"context_line":"        \u0027Name\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_d309cade","line":301,"in_reply_to":"7a740942_0f58bef9","updated":"2015-12-06 07:18:23.000000000","message":"But this is only used in this class, I think.  Sorry I didn\u0027t see your comment before it was merged. :)","commit_id":"697df67611e85116e67e65d6c28b938c172e89ef"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"880fef1df46e63a8f7cb54dc09266c9c5e5b9ea2","unresolved":false,"context_lines":[{"line_number":351,"context_line":"            external\u003dFalse,"},{"line_number":352,"context_line":"        )"},{"line_number":353,"context_line":""},{"line_number":354,"context_line":"        self.assertEqual(tuple(self.columns), columns)"},{"line_number":355,"context_line":"        self.assertEqual(self.data, list(data))"},{"line_number":356,"context_line":""},{"line_number":357,"context_line":"    def test_list_external(self, network_list):"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_cf41563b","line":354,"updated":"2015-12-04 07:19:46.000000000","message":"Same as below","commit_id":"697df67611e85116e67e65d6c28b938c172e89ef"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"880fef1df46e63a8f7cb54dc09266c9c5e5b9ea2","unresolved":false,"context_lines":[{"line_number":374,"context_line":"            external\u003dTrue,"},{"line_number":375,"context_line":"        )"},{"line_number":376,"context_line":""},{"line_number":377,"context_line":"        self.assertEqual(tuple(self.columns), columns)"},{"line_number":378,"context_line":"        self.assertEqual(self.data, list(data))"},{"line_number":379,"context_line":""},{"line_number":380,"context_line":"    def test_network_list_long(self, network_list):"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_6f52a215","line":377,"updated":"2015-12-04 07:19:46.000000000","message":"Should be assetEqual(expected, observed)\n\nThis is minor","commit_id":"697df67611e85116e67e65d6c28b938c172e89ef"},{"author":{"_account_id":14937,"name":"Tang Chen","email":"imtangchen@163.com","username":"tangchen"},"change_message_id":"df6c95c5937f5306f2a50b22e2179a3029b9a92e","unresolved":false,"context_lines":[{"line_number":374,"context_line":"            external\u003dTrue,"},{"line_number":375,"context_line":"        )"},{"line_number":376,"context_line":""},{"line_number":377,"context_line":"        self.assertEqual(tuple(self.columns), columns)"},{"line_number":378,"context_line":"        self.assertEqual(self.data, list(data))"},{"line_number":379,"context_line":""},{"line_number":380,"context_line":"    def test_network_list_long(self, network_list):"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_9313420d","line":377,"in_reply_to":"7a740942_6f52a215","updated":"2015-12-06 07:18:23.000000000","message":"Yes, self.columns is expected, and columns is observed. Sorry again I didn\u0027t see your comment before it was merged.","commit_id":"697df67611e85116e67e65d6c28b938c172e89ef"},{"author":{"_account_id":9459,"name":".Haifeng Yan","email":"yanheven@qq.com","username":"yan"},"change_message_id":"98a212118f22988184403366e7dd454dac346f84","unresolved":false,"context_lines":[{"line_number":374,"context_line":"            external\u003dTrue,"},{"line_number":375,"context_line":"        )"},{"line_number":376,"context_line":""},{"line_number":377,"context_line":"        self.assertEqual(tuple(self.columns), columns)"},{"line_number":378,"context_line":"        self.assertEqual(self.data, list(data))"},{"line_number":379,"context_line":""},{"line_number":380,"context_line":"    def test_network_list_long(self, network_list):"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_eff692d6","line":377,"in_reply_to":"7a740942_6f52a215","updated":"2015-12-04 07:30:40.000000000","message":"no wrong here.","commit_id":"697df67611e85116e67e65d6c28b938c172e89ef"}]}