GitHub PullRequestReviewCommentEvent at 2014-04-14 15:08:28
Repository: pulp
Actor: mhrivnak
Payload: {"comment":{"url":"https://api.github.com/repos/pulp/pulp/pulls/comments/11588814","id":11588814,"diff_hunk":"@@ -61,21 +62,90 @@ def __init__(self, context, name='create', description=DESC_CREATE, method=None)\n         self.add_option(OPTION_DESCRIPTION)\n         self.add_option(OPTION_NOTES)\n \n-    def run(self, **kwargs):\n+    def _parse_basic_options(self, kwargs):\n+        \"\"\"\n+        Parse the options known by this class\n+\n+        :param kwargs:  user input as provided by okaara\n+        :type  kwargs:  dict\n+\n+        :return:    tuple of repo_id, name, description, notes\n+        :rtype:     tuple\n+        \"\"\"\n+\n         # Collect input\n         repo_id = kwargs[OPTION_REPO_ID.keyword]\n         name = repo_id\n         if OPTION_NAME.keyword in kwargs:\n             name = kwargs[OPTION_NAME.keyword]\n         description = kwargs[OPTION_DESCRIPTION.keyword]\n-        notes = arg_utils.args_to_notes_dict(kwargs[OPTION_NOTES.keyword], include_none=True)\n+        if kwargs[OPTION_NOTES.keyword]:\n+            notes = arg_utils.args_to_notes_dict(kwargs[OPTION_NOTES.keyword], include_none=True)\n+        else:\n+            notes = {}","path":"client_lib/pulp/client/commands/repo/cudl.py","position":34,"original_position":34,"commit_id":"1f69c1d17d156954a009e2777796bd42789fdd96","original_commit_id":"7edc045c17b33e0da980ad379895de870b5930d9","user":{"login":"mhrivnak","id":53091,"avatar_url":"https://avatars.githubusercontent.com/u/53091?","gravatar_id":"06b29370cfbe838c3ed487118738d24e","url":"https://api.github.com/users/mhrivnak","html_url":"https://github.com/mhrivnak","followers_url":"https://api.github.com/users/mhrivnak/followers","following_url":"https://api.github.com/users/mhrivnak/following{/other_user}","gists_url":"https://api.github.com/users/mhrivnak/gists{/gist_id}","starred_url":"https://api.github.com/users/mhrivnak/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mhrivnak/subscriptions","organizations_url":"https://api.github.com/users/mhrivnak/orgs","repos_url":"https://api.github.com/users/mhrivnak/repos","events_url":"https://api.github.com/users/mhrivnak/events{/privacy}","received_events_url":"https://api.github.com/users/mhrivnak/received_events","type":"User","site_admin":false},"body":"done.","created_at":"2014-04-14T15:08:28Z","updated_at":"2014-04-14T15:08:28Z","html_url":"https://github.com/pulp/pulp/pull/897#discussion_r11588814","pull_request_url":"https://api.github.com/repos/pulp/pulp/pulls/897","_links":{"self":{"href":"https://api.github.com/repos/pulp/pulp/pulls/comments/11588814"},"html":{"href":"https://github.com/pulp/pulp/pull/897#discussion_r11588814"},"pull_request":{"href":"https://api.github.com/repos/pulp/pulp/pulls/897"}}}}