{"action":"created","comment":{"url":"https://api.github.com/repos/globus/globus-cli/pulls/comments/105441329","pull_request_review_id":26322838,"id":105441329,"diff_hunk":"@@ -6,16 +6,141 @@\n from globus_cli.services.transfer import print_json_from_iterator, get_client\n \n \n+class ISOTimeType(click.ParamType):\n+    \"\"\"\n+    Enforces time inputs to be in either YYYY-MM-DD or YYYY-MM-DD HH:MM:SS\n+    And that each value is a valid int that falls in the correct range\n+    \"\"\"\n+\n+    name = \"ISO TIME\"","path":"globus_cli/commands/task/list.py","position":10,"original_position":10,"commit_id":"ccb020b2e65b40db7dce7cf1bbe4626ed6b7c974","original_commit_id":"ccb020b2e65b40db7dce7cf1bbe4626ed6b7c974","user":{"login":"sirosen","id":1300022,"avatar_url":"https://avatars0.githubusercontent.com/u/1300022?v=3","gravatar_id":"","url":"https://api.github.com/users/sirosen","html_url":"https://github.com/sirosen","followers_url":"https://api.github.com/users/sirosen/followers","following_url":"https://api.github.com/users/sirosen/following{/other_user}","gists_url":"https://api.github.com/users/sirosen/gists{/gist_id}","starred_url":"https://api.github.com/users/sirosen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sirosen/subscriptions","organizations_url":"https://api.github.com/users/sirosen/orgs","repos_url":"https://api.github.com/users/sirosen/repos","events_url":"https://api.github.com/users/sirosen/events{/privacy}","received_events_url":"https://api.github.com/users/sirosen/received_events","type":"User","site_admin":false},"body":"Can we make this `ISO_TIME`? The Type name will sometimes show up as a metavar in help output, and we've had a past complaint that spaces in metavars were confusing.\r\n(I know it's silly, but humor me? :smile: )","created_at":"2017-03-10T17:06:05Z","updated_at":"2017-03-10T17:21:43Z","html_url":"https://github.com/globus/globus-cli/pull/228#discussion_r105441329","pull_request_url":"https://api.github.com/repos/globus/globus-cli/pulls/228","_links":{"self":{"href":"https://api.github.com/repos/globus/globus-cli/pulls/comments/105441329"},"html":{"href":"https://github.com/globus/globus-cli/pull/228#discussion_r105441329"},"pull_request":{"href":"https://api.github.com/repos/globus/globus-cli/pulls/228"}}},"pull_request":{"url":"https://api.github.com/repos/globus/globus-cli/pulls/228","id":109581328,"html_url":"https://github.com/globus/globus-cli/pull/228","diff_url":"https://github.com/globus/globus-cli/pull/228.diff","patch_url":"https://github.com/globus/globus-cli/pull/228.patch","issue_url":"https://api.github.com/repos/globus/globus-cli/issues/228","number":228,"state":"open","locked":false,"title":"Add --limit and filtering options to task list","user":{"login":"aaschaer","id":6373786,"avatar_url":"https://avatars1.githubusercontent.com/u/6373786?v=3","gravatar_id":"","url":"https://api.github.com/users/aaschaer","html_url":"https://github.com/aaschaer","followers_url":"https://api.github.com/users/aaschaer/followers","following_url":"https://api.github.com/users/aaschaer/following{/other_user}","gists_url":"https://api.github.com/users/aaschaer/gists{/gist_id}","starred_url":"https://api.github.com/users/aaschaer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aaschaer/subscriptions","organizations_url":"https://api.github.com/users/aaschaer/orgs","repos_url":"https://api.github.com/users/aaschaer/repos","events_url":"https://api.github.com/users/aaschaer/events{/privacy}","received_events_url":"https://api.github.com/users/aaschaer/received_events","type":"User","site_admin":false},"body":"Adds options to task list requested in #57\r\n\r\nI felt it was easier on the user to split filtering up into several options rather than have them try to figure out how to make a filter string and then figure out what the API errors they were getting back meant.\r\n\r\nI'm not ecstatic with the multi option method of passing more than one filter of the same kind, but it lets me use a click.Choice type for telling the user what values are allowed, and I think more typing is a lesser evil to unclear options?","created_at":"2017-03-07T22:00:51Z","updated_at":"2017-03-10T17:21:43Z","closed_at":null,"merged_at":null,"merge_commit_sha":"364c9641c313473dfacf2487cbe835705fffeaa9","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/globus/globus-cli/pulls/228/commits","review_comments_url":"https://api.github.com/repos/globus/globus-cli/pulls/228/comments","review_comment_url":"https://api.github.com/repos/globus/globus-cli/pulls/comments{/number}","comments_url":"https://api.github.com/repos/globus/globus-cli/issues/228/comments","statuses_url":"https://api.github.com/repos/globus/globus-cli/statuses/ccb020b2e65b40db7dce7cf1bbe4626ed6b7c974","head":{"label":"aaschaer:issue_57","ref":"issue_57","sha":"ccb020b2e65b40db7dce7cf1bbe4626ed6b7c974","user":{"login":"aaschaer","id":6373786,"avatar_url":"https://avatars1.githubusercontent.com/u/6373786?v=3","gravatar_id":"","url":"https://api.github.com/users/aaschaer","html_url":"https://github.com/aaschaer","followers_url":"https://api.github.com/users/aaschaer/followers","following_url":"https://api.github.com/users/aaschaer/following{/other_user}","gists_url":"https://api.github.com/users/aaschaer/gists{/gist_id}","starred_url":"https://api.github.com/users/aaschaer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aaschaer/subscriptions","organizations_url":"https://api.github.com/users/aaschaer/orgs","repos_url":"https://api.github.com/users/aaschaer/repos","events_url":"https://api.github.com/users/aaschaer/events{/privacy}","received_events_url":"https://api.github.com/users/aaschaer/received_events","type":"User","site_admin":false},"repo":{"id":78547916,"name":"globus-cli","full_name":"aaschaer/globus-cli","owner":{"login":"aaschaer","id":6373786,"avatar_url":"https://avatars1.githubusercontent.com/u/6373786?v=3","gravatar_id":"","url":"https://api.github.com/users/aaschaer","html_url":"https://github.com/aaschaer","followers_url":"https://api.github.com/users/aaschaer/followers","following_url":"https://api.github.com/users/aaschaer/following{/other_user}","gists_url":"https://api.github.com/users/aaschaer/gists{/gist_id}","starred_url":"https://api.github.com/users/aaschaer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aaschaer/subscriptions","organizations_url":"https://api.github.com/users/aaschaer/orgs","repos_url":"https://api.github.com/users/aaschaer/repos","events_url":"https://api.github.com/users/aaschaer/events{/privacy}","received_events_url":"https://api.github.com/users/aaschaer/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/aaschaer/globus-cli","description":"A Command Line Wrapper over the Globus SDK for Python","fork":true,"url":"https://api.github.com/repos/aaschaer/globus-cli","forks_url":"https://api.github.com/repos/aaschaer/globus-cli/forks","keys_url":"https://api.github.com/repos/aaschaer/globus-cli/keys{/key_id}","collaborators_url":"https://api.github.com/repos/aaschaer/globus-cli/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/aaschaer/globus-cli/teams","hooks_url":"https://api.github.com/repos/aaschaer/globus-cli/hooks","issue_events_url":"https://api.github.com/repos/aaschaer/globus-cli/issues/events{/number}","events_url":"https://api.github.com/repos/aaschaer/globus-cli/events","assignees_url":"https://api.github.com/repos/aaschaer/globus-cli/assignees{/user}","branches_url":"https://api.github.com/repos/aaschaer/globus-cli/branches{/branch}","tags_url":"https://api.github.com/repos/aaschaer/globus-cli/tags","blobs_url":"https://api.github.com/repos/aaschaer/globus-cli/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/aaschaer/globus-cli/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/aaschaer/globus-cli/git/refs{/sha}","trees_url":"https://api.github.com/repos/aaschaer/globus-cli/git/trees{/sha}","statuses_url":"https://api.github.com/repos/aaschaer/globus-cli/statuses/{sha}","languages_url":"https://api.github.com/repos/aaschaer/globus-cli/languages","stargazers_url":"https://api.github.com/repos/aaschaer/globus-cli/stargazers","contributors_url":"https://api.github.com/repos/aaschaer/globus-cli/contributors","subscribers_url":"https://api.github.com/repos/aaschaer/globus-cli/subscribers","subscription_url":"https://api.github.com/repos/aaschaer/globus-cli/subscription","commits_url":"https://api.github.com/repos/aaschaer/globus-cli/commits{/sha}","git_commits_url":"https://api.github.com/repos/aaschaer/globus-cli/git/commits{/sha}","comments_url":"https://api.github.com/repos/aaschaer/globus-cli/comments{/number}","issue_comment_url":"https://api.github.com/repos/aaschaer/globus-cli/issues/comments{/number}","contents_url":"https://api.github.com/repos/aaschaer/globus-cli/contents/{+path}","compare_url":"https://api.github.com/repos/aaschaer/globus-cli/compare/{base}...{head}","merges_url":"https://api.github.com/repos/aaschaer/globus-cli/merges","archive_url":"https://api.github.com/repos/aaschaer/globus-cli/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/aaschaer/globus-cli/downloads","issues_url":"https://api.github.com/repos/aaschaer/globus-cli/issues{/number}","pulls_url":"https://api.github.com/repos/aaschaer/globus-cli/pulls{/number}","milestones_url":"https://api.github.com/repos/aaschaer/globus-cli/milestones{/number}","notifications_url":"https://api.github.com/repos/aaschaer/globus-cli/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/aaschaer/globus-cli/labels{/name}","releases_url":"https://api.github.com/repos/aaschaer/globus-cli/releases{/id}","deployments_url":"https://api.github.com/repos/aaschaer/globus-cli/deployments","created_at":"2017-01-10T15:45:18Z","updated_at":"2017-01-10T15:45:20Z","pushed_at":"2017-03-09T17:38:09Z","git_url":"git://github.com/aaschaer/globus-cli.git","ssh_url":"git@github.com:aaschaer/globus-cli.git","clone_url":"https://github.com/aaschaer/globus-cli.git","svn_url":"https://github.com/aaschaer/globus-cli","homepage":null,"size":634,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"globus:master","ref":"master","sha":"91035f6c15644863f20bcf72aea78f13039d607d","user":{"login":"globus","id":6120321,"avatar_url":"https://avatars1.githubusercontent.com/u/6120321?v=3","gravatar_id":"","url":"https://api.github.com/users/globus","html_url":"https://github.com/globus","followers_url":"https://api.github.com/users/globus/followers","following_url":"https://api.github.com/users/globus/following{/other_user}","gists_url":"https://api.github.com/users/globus/gists{/gist_id}","starred_url":"https://api.github.com/users/globus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/globus/subscriptions","organizations_url":"https://api.github.com/users/globus/orgs","repos_url":"https://api.github.com/users/globus/repos","events_url":"https://api.github.com/users/globus/events{/privacy}","received_events_url":"https://api.github.com/users/globus/received_events","type":"Organization","site_admin":false},"repo":{"id":53438969,"name":"globus-cli","full_name":"globus/globus-cli","owner":{"login":"globus","id":6120321,"avatar_url":"https://avatars1.githubusercontent.com/u/6120321?v=3","gravatar_id":"","url":"https://api.github.com/users/globus","html_url":"https://github.com/globus","followers_url":"https://api.github.com/users/globus/followers","following_url":"https://api.github.com/users/globus/following{/other_user}","gists_url":"https://api.github.com/users/globus/gists{/gist_id}","starred_url":"https://api.github.com/users/globus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/globus/subscriptions","organizations_url":"https://api.github.com/users/globus/orgs","repos_url":"https://api.github.com/users/globus/repos","events_url":"https://api.github.com/users/globus/events{/privacy}","received_events_url":"https://api.github.com/users/globus/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/globus/globus-cli","description":"A Command Line Wrapper over the Globus SDK for Python","fork":false,"url":"https://api.github.com/repos/globus/globus-cli","forks_url":"https://api.github.com/repos/globus/globus-cli/forks","keys_url":"https://api.github.com/repos/globus/globus-cli/keys{/key_id}","collaborators_url":"https://api.github.com/repos/globus/globus-cli/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/globus/globus-cli/teams","hooks_url":"https://api.github.com/repos/globus/globus-cli/hooks","issue_events_url":"https://api.github.com/repos/globus/globus-cli/issues/events{/number}","events_url":"https://api.github.com/repos/globus/globus-cli/events","assignees_url":"https://api.github.com/repos/globus/globus-cli/assignees{/user}","branches_url":"https://api.github.com/repos/globus/globus-cli/branches{/branch}","tags_url":"https://api.github.com/repos/globus/globus-cli/tags","blobs_url":"https://api.github.com/repos/globus/globus-cli/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/globus/globus-cli/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/globus/globus-cli/git/refs{/sha}","trees_url":"https://api.github.com/repos/globus/globus-cli/git/trees{/sha}","statuses_url":"https://api.github.com/repos/globus/globus-cli/statuses/{sha}","languages_url":"https://api.github.com/repos/globus/globus-cli/languages","stargazers_url":"https://api.github.com/repos/globus/globus-cli/stargazers","contributors_url":"https://api.github.com/repos/globus/globus-cli/contributors","subscribers_url":"https://api.github.com/repos/globus/globus-cli/subscribers","subscription_url":"https://api.github.com/repos/globus/globus-cli/subscription","commits_url":"https://api.github.com/repos/globus/globus-cli/commits{/sha}","git_commits_url":"https://api.github.com/repos/globus/globus-cli/git/commits{/sha}","comments_url":"https://api.github.com/repos/globus/globus-cli/comments{/number}","issue_comment_url":"https://api.github.com/repos/globus/globus-cli/issues/comments{/number}","contents_url":"https://api.github.com/repos/globus/globus-cli/contents/{+path}","compare_url":"https://api.github.com/repos/globus/globus-cli/compare/{base}...{head}","merges_url":"https://api.github.com/repos/globus/globus-cli/merges","archive_url":"https://api.github.com/repos/globus/globus-cli/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/globus/globus-cli/downloads","issues_url":"https://api.github.com/repos/globus/globus-cli/issues{/number}","pulls_url":"https://api.github.com/repos/globus/globus-cli/pulls{/number}","milestones_url":"https://api.github.com/repos/globus/globus-cli/milestones{/number}","notifications_url":"https://api.github.com/repos/globus/globus-cli/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/globus/globus-cli/labels{/name}","releases_url":"https://api.github.com/repos/globus/globus-cli/releases{/id}","deployments_url":"https://api.github.com/repos/globus/globus-cli/deployments","created_at":"2016-03-08T19:25:41Z","updated_at":"2017-02-28T17:05:16Z","pushed_at":"2017-03-09T18:31:39Z","git_url":"git://github.com/globus/globus-cli.git","ssh_url":"git@github.com:globus/globus-cli.git","clone_url":"https://github.com/globus/globus-cli.git","svn_url":"https://github.com/globus/globus-cli","homepage":null,"size":628,"stargazers_count":8,"watchers_count":8,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":5,"mirror_url":null,"open_issues_count":59,"forks":5,"open_issues":59,"watchers":8,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/globus/globus-cli/pulls/228"},"html":{"href":"https://github.com/globus/globus-cli/pull/228"},"issue":{"href":"https://api.github.com/repos/globus/globus-cli/issues/228"},"comments":{"href":"https://api.github.com/repos/globus/globus-cli/issues/228/comments"},"review_comments":{"href":"https://api.github.com/repos/globus/globus-cli/pulls/228/comments"},"review_comment":{"href":"https://api.github.com/repos/globus/globus-cli/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/globus/globus-cli/pulls/228/commits"},"statuses":{"href":"https://api.github.com/repos/globus/globus-cli/statuses/ccb020b2e65b40db7dce7cf1bbe4626ed6b7c974"}}}}
{"actor":{"display_login":"sirosen"}}