)]}'
{"HACKING.rst":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"8d9f4d99d7ebf71c4b1f6efe19dcc58d08f97da2","unresolved":false,"context_lines":[{"line_number":20,"context_line":"- [C302] six.text_type should be used instead of unicode"},{"line_number":21,"context_line":"- [C303] Ensure that there are no \u0027print()\u0027 statements in code that is being committed."},{"line_number":22,"context_line":"- [C304] Enforce no use of LOG.audit messages.  LOG.info should be used instead."},{"line_number":23,"context_line":"- [C305] Prevent use of deprecated contextlib.nested."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"General"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa53dc9_77eee6ed","line":23,"updated":"2015-04-06 19:35:42.000000000","message":"N339 was actually the Nova hacking check. That patch is still under review.\n\nI think that is a gap. If that review ends up getting a merge conflict and needs to bump up their number then our use of N339 is no longer valid even though it is already merged.\n\nSo I think this is good you are changing this to a C number. Just pointing out that any use of non-C numbering has some risk without a cross-project registration for each number. We could potentially go back after the Nova one has merged and math them up, but probably safer and better to keep things with a C numbering.","commit_id":"d0c6cd500b37ec557cfda01c44bdf981b81d8047"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"c2be71548ec307471dd02c0960a64017ccd33b60","unresolved":false,"context_lines":[{"line_number":20,"context_line":"- [C302] six.text_type should be used instead of unicode"},{"line_number":21,"context_line":"- [C303] Ensure that there are no \u0027print()\u0027 statements in code that is being committed."},{"line_number":22,"context_line":"- [C304] Enforce no use of LOG.audit messages.  LOG.info should be used instead."},{"line_number":23,"context_line":"- [C305] Prevent use of deprecated contextlib.nested."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"General"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa53dc9_15f61bd7","line":23,"in_reply_to":"9aa53dc9_26bec0d4","updated":"2015-04-08 15:17:10.000000000","message":"Let me know if you need any help with that.","commit_id":"d0c6cd500b37ec557cfda01c44bdf981b81d8047"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"4102322632bb8d249c91d15fc887a8af21896b29","unresolved":false,"context_lines":[{"line_number":20,"context_line":"- [C302] six.text_type should be used instead of unicode"},{"line_number":21,"context_line":"- [C303] Ensure that there are no \u0027print()\u0027 statements in code that is being committed."},{"line_number":22,"context_line":"- [C304] Enforce no use of LOG.audit messages.  LOG.info should be used instead."},{"line_number":23,"context_line":"- [C305] Prevent use of deprecated contextlib.nested."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"General"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa53dc9_a0799d95","line":23,"in_reply_to":"9aa53dc9_77eee6ed","updated":"2015-04-07 16:52:34.000000000","message":"+1","commit_id":"d0c6cd500b37ec557cfda01c44bdf981b81d8047"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"f21798aad774eacd931d8905a79ed7a2168304df","unresolved":false,"context_lines":[{"line_number":20,"context_line":"- [C302] six.text_type should be used instead of unicode"},{"line_number":21,"context_line":"- [C303] Ensure that there are no \u0027print()\u0027 statements in code that is being committed."},{"line_number":22,"context_line":"- [C304] Enforce no use of LOG.audit messages.  LOG.info should be used instead."},{"line_number":23,"context_line":"- [C305] Prevent use of deprecated contextlib.nested."},{"line_number":24,"context_line":""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"General"}],"source_content_type":"text/x-rst","patch_set":1,"id":"9aa53dc9_26bec0d4","line":23,"in_reply_to":"9aa53dc9_77eee6ed","updated":"2015-04-07 19:43:04.000000000","message":"Sounds good.  On a related note, I have a bunch of other work in this area I would like to do as soon as Liberty opens up.  So, that will be part of the work.\n\n/me lives for the grunt work.  :-)","commit_id":"d0c6cd500b37ec557cfda01c44bdf981b81d8047"}]}