)]}'
{"quantumclient/quantum/v2_0/__init__.py":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"8722219b24766ea9c9030848edf35783c77a1446","unresolved":false,"context_lines":[{"line_number":322,"context_line":"        return parser"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":325,"context_line":"        pass"},{"line_number":326,"context_line":""},{"line_number":327,"context_line":"    def args2body(self, parsed_args):"},{"line_number":328,"context_line":"        return {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FOYw%3D","line":325,"updated":"2013-02-01 12:23:02.000000000","message":"It took me a few minutes to understand why you did it here.\nCan you add a comment to state that this method and the one below are mean to be overriden in subclasses.\n\nAlso consider if we have a case for moving them one level above in the hierarchy (in quantumcommand)","commit_id":"bd095eb61aabb1b4ad2d618cdc76b4400a20f116"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"unresolved":false,"context_lines":[{"line_number":322,"context_line":"        return parser"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":325,"context_line":"        pass"},{"line_number":326,"context_line":""},{"line_number":327,"context_line":"    def args2body(self, parsed_args):"},{"line_number":328,"context_line":"        return {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FNr0%3D","line":325,"in_reply_to":"AAAAM3%2F%2FN1k%3D","updated":"2013-02-01 22:08:50.000000000","message":"indeed.","commit_id":"bd095eb61aabb1b4ad2d618cdc76b4400a20f116"},{"author":{"_account_id":4395,"name":"Aaron Rosen","email":"aaronorosen@gmail.com","username":"arosen"},"change_message_id":"323fd2ab928ea9e22a70f8a148276d622afedc76","unresolved":false,"context_lines":[{"line_number":322,"context_line":"        return parser"},{"line_number":323,"context_line":""},{"line_number":324,"context_line":"    def add_known_arguments(self, parser):"},{"line_number":325,"context_line":"        pass"},{"line_number":326,"context_line":""},{"line_number":327,"context_line":"    def args2body(self, parsed_args):"},{"line_number":328,"context_line":"        return {}"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FN1k%3D","line":325,"in_reply_to":"AAAAM3%2F%2FOYw%3D","updated":"2013-02-01 20:24:14.000000000","message":"Sure, the reason I needed to do this is because all update commands inherit from this class (UpdateCommand). Therefore, i need to provide a base class implementation if i want to have the derived class override it. \n\nIn my opinion a comment here isn\u0027t really needed since this is the way that CreateCommand,  ListCommand,  QuantumCommand already work. They all provide a base method that either call pass or return. \n\nThat is a good idea for moving args2body and add_know_arguments to QuantumCommand. I\u0027ll do that. Doing this also will remove the need to add a comment here since that line won\u0027t be here anymore :)","commit_id":"bd095eb61aabb1b4ad2d618cdc76b4400a20f116"}]}