GitHub PullRequestEvent at 2014-02-07 20:39:40
Repository: aws-cli
Actor: jamesls
Payload: {"action":"opened","number":639,"pull_request":{"url":"https://api.github.com/repos/aws/aws-cli/pulls/639","id":12328548,"html_url":"https://github.com/aws/aws-cli/pull/639","diff_url":"https://github.com/aws/aws-cli/pull/639.diff","patch_url":"https://github.com/aws/aws-cli/pull/639.patch","issue_url":"https://api.github.com/repos/aws/aws-cli/issues/639","number":639,"state":"open","title":"Show better error message for invalid JSON","user":{"login":"jamesls","id":368057,"avatar_url":"https://gravatar.com/avatar/a8cb6a42bfa3881315aafa5e027390e3?d=https%3A%2F%2Fidenticons.github.com%2Fc6b13e1c274e3697b6559ac59936b3f7.png&r=x","gravatar_id":"a8cb6a42bfa3881315aafa5e027390e3","url":"https://api.github.com/users/jamesls","html_url":"https://github.com/jamesls","followers_url":"https://api.github.com/users/jamesls/followers","following_url":"https://api.github.com/users/jamesls/following{/other_user}","gists_url":"https://api.github.com/users/jamesls/gists{/gist_id}","starred_url":"https://api.github.com/users/jamesls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jamesls/subscriptions","organizations_url":"https://api.github.com/users/jamesls/orgs","repos_url":"https://api.github.com/users/jamesls/repos","events_url":"https://api.github.com/users/jamesls/events{/privacy}","received_events_url":"https://api.github.com/users/jamesls/received_events","type":"User","site_admin":false},"body":"\r\nEspecially for list params, their error messages were not helpful.\r\nError messages now include the actual value itself, as we have seen\r\ncases where customers specify valid JSON but the shell they're using\r\nwill process arguments and strip out characters such as quotes.\r\nBy showing the actual JSON contents it will make this more obvious.\r\n\r\nBefore:\r\n```\r\n  The value for parameter \"--\" must be JSON or path to file.\r\n```\r\nAfter:\r\n```\r\n  Error parsing parameter --block-device-mappings: Invalid JSON:\r\n  [{ ...BAD JSON CONTENTS ...}]\r\n```","created_at":"2014-02-07T20:39:40Z","updated_at":"2014-02-07T20:39:40Z","closed_at":null,"merged_at":null,"merge_commit_sha":null,"assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/aws/aws-cli/pulls/639/commits","review_comments_url":"https://api.github.com/repos/aws/aws-cli/pulls/639/comments","review_comment_url":"https://api.github.com/repos/aws/aws-cli/pulls/comments/{number}","comments_url":"https://api.github.com/repos/aws/aws-cli/issues/639/comments","statuses_url":"https://api.github.com/repos/aws/aws-cli/statuses/2a95d9bdefbefeeee4349821cdcb78f6136dbedc","head":{"label":"jamesls:multi-json-arg","ref":"multi-json-arg","sha":"2a95d9bdefbefeeee4349821cdcb78f6136dbedc","user":{"login":"jamesls","id":368057,"avatar_url":"https://gravatar.com/avatar/a8cb6a42bfa3881315aafa5e027390e3?d=https%3A%2F%2Fidenticons.github.com%2Fc6b13e1c274e3697b6559ac59936b3f7.png&r=x","gravatar_id":"a8cb6a42bfa3881315aafa5e027390e3","url":"https://api.github.com/users/jamesls","html_url":"https://github.com/jamesls","followers_url":"https://api.github.com/users/jamesls/followers","following_url":"https://api.github.com/users/jamesls/following{/other_user}","gists_url":"https://api.github.com/users/jamesls/gists{/gist_id}","starred_url":"https://api.github.com/users/jamesls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jamesls/subscriptions","organizations_url":"https://api.github.com/users/jamesls/orgs","repos_url":"https://api.github.com/users/jamesls/repos","events_url":"https://api.github.com/users/jamesls/events{/privacy}","received_events_url":"https://api.github.com/users/jamesls/received_events","type":"User","site_admin":false},"repo":{"id":8471236,"name":"aws-cli","full_name":"jamesls/aws-cli","owner":{"login":"jamesls","id":368057,"avatar_url":"https://gravatar.com/avatar/a8cb6a42bfa3881315aafa5e027390e3?d=https%3A%2F%2Fidenticons.github.com%2Fc6b13e1c274e3697b6559ac59936b3f7.png&r=x","gravatar_id":"a8cb6a42bfa3881315aafa5e027390e3","url":"https://api.github.com/users/jamesls","html_url":"https://github.com/jamesls","followers_url":"https://api.github.com/users/jamesls/followers","following_url":"https://api.github.com/users/jamesls/following{/other_user}","gists_url":"https://api.github.com/users/jamesls/gists{/gist_id}","starred_url":"https://api.github.com/users/jamesls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jamesls/subscriptions","organizations_url":"https://api.github.com/users/jamesls/orgs","repos_url":"https://api.github.com/users/jamesls/repos","events_url":"https://api.github.com/users/jamesls/events{/privacy}","received_events_url":"https://api.github.com/users/jamesls/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/jamesls/aws-cli","description":"Universal Command Line Interface for Amazon Web Services","fork":true,"url":"https://api.github.com/repos/jamesls/aws-cli","forks_url":"https://api.github.com/repos/jamesls/aws-cli/forks","keys_url":"https://api.github.com/repos/jamesls/aws-cli/keys{/key_id}","collaborators_url":"https://api.github.com/repos/jamesls/aws-cli/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/jamesls/aws-cli/teams","hooks_url":"https://api.github.com/repos/jamesls/aws-cli/hooks","issue_events_url":"https://api.github.com/repos/jamesls/aws-cli/issues/events{/number}","events_url":"https://api.github.com/repos/jamesls/aws-cli/events","assignees_url":"https://api.github.com/repos/jamesls/aws-cli/assignees{/user}","branches_url":"https://api.github.com/repos/jamesls/aws-cli/branches{/branch}","tags_url":"https://api.github.com/repos/jamesls/aws-cli/tags","blobs_url":"https://api.github.com/repos/jamesls/aws-cli/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/jamesls/aws-cli/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/jamesls/aws-cli/git/refs{/sha}","trees_url":"https://api.github.com/repos/jamesls/aws-cli/git/trees{/sha}","statuses_url":"https://api.github.com/repos/jamesls/aws-cli/statuses/{sha}","languages_url":"https://api.github.com/repos/jamesls/aws-cli/languages","stargazers_url":"https://api.github.com/repos/jamesls/aws-cli/stargazers","contributors_url":"https://api.github.com/repos/jamesls/aws-cli/contributors","subscribers_url":"https://api.github.com/repos/jamesls/aws-cli/subscribers","subscription_url":"https://api.github.com/repos/jamesls/aws-cli/subscription","commits_url":"https://api.github.com/repos/jamesls/aws-cli/commits{/sha}","git_commits_url":"https://api.github.com/repos/jamesls/aws-cli/git/commits{/sha}","comments_url":"https://api.github.com/repos/jamesls/aws-cli/comments{/number}","issue_comment_url":"https://api.github.com/repos/jamesls/aws-cli/issues/comments/{number}","contents_url":"https://api.github.com/repos/jamesls/aws-cli/contents/{+path}","compare_url":"https://api.github.com/repos/jamesls/aws-cli/compare/{base}...{head}","merges_url":"https://api.github.com/repos/jamesls/aws-cli/merges","archive_url":"https://api.github.com/repos/jamesls/aws-cli/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/jamesls/aws-cli/downloads","issues_url":"https://api.github.com/repos/jamesls/aws-cli/issues{/number}","pulls_url":"https://api.github.com/repos/jamesls/aws-cli/pulls{/number}","milestones_url":"https://api.github.com/repos/jamesls/aws-cli/milestones{/number}","notifications_url":"https://api.github.com/repos/jamesls/aws-cli/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/jamesls/aws-cli/labels{/name}","releases_url":"https://api.github.com/repos/jamesls/aws-cli/releases{/id}","created_at":"2013-02-28T03:36:51Z","updated_at":"2014-02-07T20:39:40Z","pushed_at":"2014-02-07T20:36:32Z","git_url":"git://github.com/jamesls/aws-cli.git","ssh_url":"git@github.com:jamesls/aws-cli.git","clone_url":"https://github.com/jamesls/aws-cli.git","svn_url":"https://github.com/jamesls/aws-cli","homepage":null,"size":5401,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"develop","master_branch":"develop"}},"base":{"label":"aws:develop","ref":"develop","sha":"0877312cdad3cd01d6e624a691a3e5b8047cfabd","user":{"login":"aws","id":2232217,"avatar_url":"https://gravatar.com/avatar/b66042c7e9e8ae1a0fe1c0c3245d3d91?d=https%3A%2F%2Fidenticons.github.com%2Fade83eb3a0f545d52ee4a72176faff2a.png&r=x","gravatar_id":"b66042c7e9e8ae1a0fe1c0c3245d3d91","url":"https://api.github.com/users/aws","html_url":"https://github.com/aws","followers_url":"https://api.github.com/users/aws/followers","following_url":"https://api.github.com/users/aws/following{/other_user}","gists_url":"https://api.github.com/users/aws/gists{/gist_id}","starred_url":"https://api.github.com/users/aws/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aws/subscriptions","organizations_url":"https://api.github.com/users/aws/orgs","repos_url":"https://api.github.com/users/aws/repos","events_url":"https://api.github.com/users/aws/events{/privacy}","received_events_url":"https://api.github.com/users/aws/received_events","type":"Organization","site_admin":false},"repo":{"id":6780767,"name":"aws-cli","full_name":"aws/aws-cli","owner":{"login":"aws","id":2232217,"avatar_url":"https://gravatar.com/avatar/b66042c7e9e8ae1a0fe1c0c3245d3d91?d=https%3A%2F%2Fidenticons.github.com%2Fade83eb3a0f545d52ee4a72176faff2a.png&r=x","gravatar_id":"b66042c7e9e8ae1a0fe1c0c3245d3d91","url":"https://api.github.com/users/aws","html_url":"https://github.com/aws","followers_url":"https://api.github.com/users/aws/followers","following_url":"https://api.github.com/users/aws/following{/other_user}","gists_url":"https://api.github.com/users/aws/gists{/gist_id}","starred_url":"https://api.github.com/users/aws/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aws/subscriptions","organizations_url":"https://api.github.com/users/aws/orgs","repos_url":"https://api.github.com/users/aws/repos","events_url":"https://api.github.com/users/aws/events{/privacy}","received_events_url":"https://api.github.com/users/aws/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/aws/aws-cli","description":"Universal Command Line Interface for Amazon Web Services","fork":false,"url":"https://api.github.com/repos/aws/aws-cli","forks_url":"https://api.github.com/repos/aws/aws-cli/forks","keys_url":"https://api.github.com/repos/aws/aws-cli/keys{/key_id}","collaborators_url":"https://api.github.com/repos/aws/aws-cli/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/aws/aws-cli/teams","hooks_url":"https://api.github.com/repos/aws/aws-cli/hooks","issue_events_url":"https://api.github.com/repos/aws/aws-cli/issues/events{/number}","events_url":"https://api.github.com/repos/aws/aws-cli/events","assignees_url":"https://api.github.com/repos/aws/aws-cli/assignees{/user}","branches_url":"https://api.github.com/repos/aws/aws-cli/branches{/branch}","tags_url":"https://api.github.com/repos/aws/aws-cli/tags","blobs_url":"https://api.github.com/repos/aws/aws-cli/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/aws/aws-cli/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/aws/aws-cli/git/refs{/sha}","trees_url":"https://api.github.com/repos/aws/aws-cli/git/trees{/sha}","statuses_url":"https://api.github.com/repos/aws/aws-cli/statuses/{sha}","languages_url":"https://api.github.com/repos/aws/aws-cli/languages","stargazers_url":"https://api.github.com/repos/aws/aws-cli/stargazers","contributors_url":"https://api.github.com/repos/aws/aws-cli/contributors","subscribers_url":"https://api.github.com/repos/aws/aws-cli/subscribers","subscription_url":"https://api.github.com/repos/aws/aws-cli/subscription","commits_url":"https://api.github.com/repos/aws/aws-cli/commits{/sha}","git_commits_url":"https://api.github.com/repos/aws/aws-cli/git/commits{/sha}","comments_url":"https://api.github.com/repos/aws/aws-cli/comments{/number}","issue_comment_url":"https://api.github.com/repos/aws/aws-cli/issues/comments/{number}","contents_url":"https://api.github.com/repos/aws/aws-cli/contents/{+path}","compare_url":"https://api.github.com/repos/aws/aws-cli/compare/{base}...{head}","merges_url":"https://api.github.com/repos/aws/aws-cli/merges","archive_url":"https://api.github.com/repos/aws/aws-cli/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/aws/aws-cli/downloads","issues_url":"https://api.github.com/repos/aws/aws-cli/issues{/number}","pulls_url":"https://api.github.com/repos/aws/aws-cli/pulls{/number}","milestones_url":"https://api.github.com/repos/aws/aws-cli/milestones{/number}","notifications_url":"https://api.github.com/repos/aws/aws-cli/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/aws/aws-cli/labels{/name}","releases_url":"https://api.github.com/repos/aws/aws-cli/releases{/id}","created_at":"2012-11-20T16:07:36Z","updated_at":"2014-02-06T19:52:29Z","pushed_at":"2014-02-06T19:52:29Z","git_url":"git://github.com/aws/aws-cli.git","ssh_url":"git@github.com:aws/aws-cli.git","clone_url":"https://github.com/aws/aws-cli.git","svn_url":"https://github.com/aws/aws-cli","homepage":null,"size":8986,"stargazers_count":1684,"watchers_count":1684,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"forks_count":175,"mirror_url":null,"open_issues_count":83,"forks":175,"open_issues":83,"watchers":1684,"default_branch":"develop","master_branch":"develop"}},"_links":{"self":{"href":"https://api.github.com/repos/aws/aws-cli/pulls/639"},"html":{"href":"https://github.com/aws/aws-cli/pull/639"},"issue":{"href":"https://api.github.com/repos/aws/aws-cli/issues/639"},"comments":{"href":"https://api.github.com/repos/aws/aws-cli/issues/639/comments"},"review_comments":{"href":"https://api.github.com/repos/aws/aws-cli/pulls/639/comments"},"review_comment":{"href":"https://api.github.com/repos/aws/aws-cli/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/aws/aws-cli/pulls/639/commits"},"statuses":{"href":"https://api.github.com/repos/aws/aws-cli/statuses/2a95d9bdefbefeeee4349821cdcb78f6136dbedc"}},"merged":false,"mergeable":null,"mergeable_state":"unknown","merged_by":null,"comments":0,"review_comments":0,"commits":2,"additions":53,"deletions":20,"changed_files":4}}