)]}'
{"HACKING":[{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"55c289065ceae6ff3d089780c743b38cdfc387d7","unresolved":false,"context_lines":[{"line_number":13,"context_line":"- Develop tests with parallel execution in mind"},{"line_number":14,"context_line":"- Keep the scope of any test granular"},{"line_number":15,"context_line":"- Set skip conditions for tests that can only be run in"},{"line_number":16,"context_line":"  certain environments   "},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Test Data/Configuration"},{"line_number":19,"context_line":"-----------------------"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACX%2F%2F6vI%3D","line":16,"updated":"2011-10-25 16:30:34.000000000","message":"Whitespace at end of line here....","commit_id":"737830fbe68d256ee650873669917bc0ee0e0fc2"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"55c289065ceae6ff3d089780c743b38cdfc387d7","unresolved":false,"context_lines":[{"line_number":33,"context_line":"Imports"},{"line_number":34,"context_line":"-------"},{"line_number":35,"context_line":"- Do not import objects, only modules"},{"line_number":36,"context_line":"- Do not import more than one module per line"},{"line_number":37,"context_line":"- Do not make relative imports"},{"line_number":38,"context_line":"- Order your imports by the full module path"},{"line_number":39,"context_line":"- Organize your imports according to the following template"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACX%2F%2F6vE%3D","line":36,"updated":"2011-10-25 16:30:34.000000000","message":"I think the above two are silly.","commit_id":"737830fbe68d256ee650873669917bc0ee0e0fc2"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"066b8eccdecec7158acfd970438f94550b14cfd1","unresolved":false,"context_lines":[{"line_number":33,"context_line":"Imports"},{"line_number":34,"context_line":"-------"},{"line_number":35,"context_line":"- Do not import objects, only modules"},{"line_number":36,"context_line":"- Do not import more than one module per line"},{"line_number":37,"context_line":"- Do not make relative imports"},{"line_number":38,"context_line":"- Order your imports by the full module path"},{"line_number":39,"context_line":"- Organize your imports according to the following template"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACX%2F%2F6Lc%3D","line":36,"in_reply_to":"AAAACX%2F%2F6vE%3D","updated":"2011-10-27 16:50:47.000000000","message":"I do not think they are silly. I think it\u0027s helpful to preserve the namespace of objects not defined within the current module.","commit_id":"737830fbe68d256ee650873669917bc0ee0e0fc2"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"55c289065ceae6ff3d089780c743b38cdfc387d7","unresolved":false,"context_lines":[{"line_number":97,"context_line":"  :returns: description of the return value"},{"line_number":98,"context_line":"  :raises: AttributeError, KeyError"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"  \"\"\""},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Calling Methods"},{"line_number":103,"context_line":"---------------"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACX%2F%2F6vA%3D","line":100,"updated":"2011-10-25 16:30:34.000000000","message":"PLEASE can we NOT use the Nova docstring style :) I much prefer not starting the comment on the same line as the opening \"\"\" and not requiring an artificial 80-character long \"brief description\" along with a longer description. Also, the extra newline at the end is retarded IMHO.","commit_id":"737830fbe68d256ee650873669917bc0ee0e0fc2"},{"author":{"_account_id":1132,"name":"Brian Waldon","email":"brian@waldon.cc","username":"bcwaldon"},"change_message_id":"066b8eccdecec7158acfd970438f94550b14cfd1","unresolved":false,"context_lines":[{"line_number":97,"context_line":"  :returns: description of the return value"},{"line_number":98,"context_line":"  :raises: AttributeError, KeyError"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"  \"\"\""},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"Calling Methods"},{"line_number":103,"context_line":"---------------"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACX%2F%2F6LY%3D","line":100,"in_reply_to":"AAAACX%2F%2F6vA%3D","updated":"2011-10-27 16:50:47.000000000","message":"++","commit_id":"737830fbe68d256ee650873669917bc0ee0e0fc2"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"55c289065ceae6ff3d089780c743b38cdfc387d7","unresolved":false,"context_lines":[{"line_number":102,"context_line":"Calling Methods"},{"line_number":103,"context_line":"---------------"},{"line_number":104,"context_line":"  Calls to methods 80 characters or longer should format each argument with "},{"line_number":105,"context_line":"  newlines. This is not a requirement, but a guideline."},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"    unnecessarily_long_function_name(\u0027string one\u0027,"},{"line_number":108,"context_line":"                                     \u0027string two\u0027,"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACX%2F%2F6u8%3D","line":105,"updated":"2011-10-25 16:30:34.000000000","message":"whitespace at end :)","commit_id":"737830fbe68d256ee650873669917bc0ee0e0fc2"},{"author":{"_account_id":7,"name":"Jay Pipes","email":"jaypipes@gmail.com","username":"jaypipes"},"change_message_id":"55c289065ceae6ff3d089780c743b38cdfc387d7","unresolved":false,"context_lines":[{"line_number":122,"context_line":"        \u0027two\u0027: 2,"},{"line_number":123,"context_line":"        \u0027twenty four\u0027: 24,"},{"line_number":124,"context_line":"    }"},{"line_number":125,"context_line":"        "},{"line_number":126,"context_line":"    object_one.call_a_method(\u0027string three\u0027,"},{"line_number":127,"context_line":"                             \u0027string four\u0027,"},{"line_number":128,"context_line":"                             kwarg1\u003dlist_of_strings,"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAACX%2F%2F6u4%3D","line":125,"updated":"2011-10-25 16:30:34.000000000","message":"Is that a TAB?! ;)","commit_id":"737830fbe68d256ee650873669917bc0ee0e0fc2"}]}
