)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"25a5a980f0ea48bc90a5e22ffd5cd0a6169bdffc","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Bertrand Lallau \u003cbertrand.lallau@thalesgroup.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-11-02 17:26:56 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use assertIs(Not)None instead of assert(Not)Equal"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Instead of using assert(Not)Equal(None, ***), developers should"},{"line_number":10,"context_line":"use assertIs(Not)None(***) to have more clear messages in case of failure."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"fa80f949_d8727e7c","line":7,"updated":"2015-11-04 04:00:38.000000000","message":"assertNotEqual doesn\u0027t seem to be covered by the change.","commit_id":"25eb3379df033e48172673bca7a3350b399c0bf4"}],"HACKING.rst":[{"author":{"_account_id":16237,"name":"venkatamahesh","email":"venkatamaheshkotha@gmail.com","username":"venkatamahesh"},"change_message_id":"533bc69b8a73c27983835e897e3c5e8aa85da7a3","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Octavia Specific Commandments"},{"line_number":10,"context_line":"-----------------------------"},{"line_number":11,"context_line":"- [N318] Change assertEqual(A, None) or assertEqual(None, A) by optimal assert like"},{"line_number":12,"context_line":"  assertIsNone(A)"},{"line_number":13,"context_line":"- [N320] Validate that LOG messages, except debug ones, have translations"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Creating Unit Tests"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a26ad4f_8e52a1dc","line":12,"updated":"2015-10-27 17:51:46.000000000","message":"for assertIsNotNone also this line is ok ? or should document separate for it ?","commit_id":"c8c295b5fb6c646b14a8d69c3937c49ffe9b6e90"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"ae3ef3c5f3fa5026902af498747d7dd2c99f0d32","unresolved":false,"context_lines":[{"line_number":6,"context_line":"- We follow the OpenStack Style Commandments:"},{"line_number":7,"context_line":"  http://docs.openstack.org/developer/hacking/"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Octavia Specific Commandments"},{"line_number":10,"context_line":"-----------------------------"},{"line_number":11,"context_line":"- [N316] Change assertTrue(isinstance(A, B)) by optimal assert like"},{"line_number":12,"context_line":"  assertIsInstance(A, B)."}],"source_content_type":"text/x-rst","patch_set":4,"id":"1a26ad4f_b8ce432c","line":9,"updated":"2015-11-02 05:55:56.000000000","message":"Nxxx are neutron, arent\u0027 they?","commit_id":"14eb5b139b20a3c11804c244c4eaf68d52f52ddd"},{"author":{"_account_id":6854,"name":"YAMAMOTO Takashi","email":"yamamoto@midokura.com","username":"yamamoto"},"change_message_id":"6a52d65d1f5ce610b53ba6738abe3db5a6d8ccac","unresolved":false,"context_lines":[{"line_number":11,"context_line":"- [N316] Change assertTrue(isinstance(A, B)) by optimal assert like"},{"line_number":12,"context_line":"  assertIsInstance(A, B)."},{"line_number":13,"context_line":"- [O318] Change assertEqual(A, None) or assertEqual(None, A) by optimal"},{"line_number":14,"context_line":"  assert like assertIsNone(A)"},{"line_number":15,"context_line":"- [N320] Validate that LOG messages, except debug ones, have translations"},{"line_number":16,"context_line":"- [N334] Change assertTrue/False(A in/not in B, message) to the more specific"},{"line_number":17,"context_line":"  assertIn/NotIn(A, B, message)"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1a26ad4f_f64efa6d","line":14,"updated":"2015-11-02 15:44:42.000000000","message":"where is the code?","commit_id":"aa62a99ccac5752d456b2c64d8f98d5765b72e4f"}],"octavia/tests/functional/api/v1/test_pool.py":[{"author":{"_account_id":10273,"name":"Adam Harwell","email":"flux.adam@gmail.com","username":"rm_you"},"change_message_id":"137e8f2bf61f5d1b1d9be55dc98fdbdbe928127e","unresolved":false,"context_lines":[{"line_number":343,"context_line":"                                            self.listener.get(\u0027id\u0027),"},{"line_number":344,"context_line":"                                            constants.PENDING_UPDATE,"},{"line_number":345,"context_line":"                                            constants.ONLINE)"},{"line_number":346,"context_line":"        self.assertIsNotNone(api_pool.get(\u0027session_persistence\u0027))"},{"line_number":347,"context_line":""},{"line_number":348,"context_line":"    def test_create_when_lb_pending_update(self):"},{"line_number":349,"context_line":"        self.put(self.LB_PATH.format(lb_id\u003dself.lb.get(\u0027id\u0027)),"}],"source_content_type":"text/x-python","patch_set":7,"id":"fa80f949_760fd6e1","line":346,"updated":"2015-11-04 12:03:46.000000000","message":"So originally you were planning to add this to the checks too, but didn\u0027t? I guess I don\u0027t mind you fixing it here while you\u0027re at it, but are you planning to add this to checks eventually as well?","commit_id":"8f735344846bdaba9f55f975ea4a8d93d15e589b"}]}