{"action":"created","comment":{"url":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/comments/120410432","pull_request_review_id":42371027,"id":120410432,"diff_hunk":"@@ -147,6 +147,16 @@ func (command *Command) Parse(rawCommand string) (bool, error) {\n \t\t\treturn true, command.Err\n \t\t}\n \n+\t\t// Check if parameter types are correct\n+\t\tfor i, _ := range opts.Params {\n+\t\t\tif opts.ParamInteger[i] {\n+\t\t\t\t_, err := strconv.Atoi(parts[i+1])","path":"architecture/protocol.go","position":7,"original_position":7,"commit_id":"fe80bb451270148ba9d025da952d072de11da4b7","original_commit_id":"fe80bb451270148ba9d025da952d072de11da4b7","user":{"login":"stephenwithav","id":54563,"avatar_url":"https://avatars2.githubusercontent.com/u/54563?v=3","gravatar_id":"","url":"https://api.github.com/users/stephenwithav","html_url":"https://github.com/stephenwithav","followers_url":"https://api.github.com/users/stephenwithav/followers","following_url":"https://api.github.com/users/stephenwithav/following{/other_user}","gists_url":"https://api.github.com/users/stephenwithav/gists{/gist_id}","starred_url":"https://api.github.com/users/stephenwithav/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stephenwithav/subscriptions","organizations_url":"https://api.github.com/users/stephenwithav/orgs","repos_url":"https://api.github.com/users/stephenwithav/repos","events_url":"https://api.github.com/users/stephenwithav/events{/privacy}","received_events_url":"https://api.github.com/users/stephenwithav/received_events","type":"User","site_admin":false},"body":"@marekyggdrasil check out #77 for a solution to parsing/validating just once.  @vimukthi-git suggested using `interface{}` instead of adding everything to `Command`, which is doable.\r\n\r\nI'm going to work on #39 after lunch.  Do you want to take a crack at refactoring #77--or incorporating it into this PR?  If not, I'll handle this after #39.","created_at":"2017-06-06T16:27:21Z","updated_at":"2017-06-06T16:27:21Z","html_url":"https://github.com/Beanstalkg/beanstalkg/pull/74#discussion_r120410432","pull_request_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/74","_links":{"self":{"href":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/comments/120410432"},"html":{"href":"https://github.com/Beanstalkg/beanstalkg/pull/74#discussion_r120410432"},"pull_request":{"href":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/74"}}},"pull_request":{"url":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/74","id":123835798,"html_url":"https://github.com/Beanstalkg/beanstalkg/pull/74","diff_url":"https://github.com/Beanstalkg/beanstalkg/pull/74.diff","patch_url":"https://github.com/Beanstalkg/beanstalkg/pull/74.patch","issue_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/issues/74","number":74,"state":"open","locked":false,"title":"validation of command parameters","user":{"login":"marekyggdrasil","id":8202674,"avatar_url":"https://avatars3.githubusercontent.com/u/8202674?v=3","gravatar_id":"","url":"https://api.github.com/users/marekyggdrasil","html_url":"https://github.com/marekyggdrasil","followers_url":"https://api.github.com/users/marekyggdrasil/followers","following_url":"https://api.github.com/users/marekyggdrasil/following{/other_user}","gists_url":"https://api.github.com/users/marekyggdrasil/gists{/gist_id}","starred_url":"https://api.github.com/users/marekyggdrasil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marekyggdrasil/subscriptions","organizations_url":"https://api.github.com/users/marekyggdrasil/orgs","repos_url":"https://api.github.com/users/marekyggdrasil/repos","events_url":"https://api.github.com/users/marekyggdrasil/events{/privacy}","received_events_url":"https://api.github.com/users/marekyggdrasil/received_events","type":"User","site_admin":false},"body":"","created_at":"2017-06-03T09:06:35Z","updated_at":"2017-06-06T16:27:21Z","closed_at":null,"merged_at":null,"merge_commit_sha":"4b0bf38af0b8fd50f7cf5f1f194214e655b211e3","assignee":{"login":"marekyggdrasil","id":8202674,"avatar_url":"https://avatars3.githubusercontent.com/u/8202674?v=3","gravatar_id":"","url":"https://api.github.com/users/marekyggdrasil","html_url":"https://github.com/marekyggdrasil","followers_url":"https://api.github.com/users/marekyggdrasil/followers","following_url":"https://api.github.com/users/marekyggdrasil/following{/other_user}","gists_url":"https://api.github.com/users/marekyggdrasil/gists{/gist_id}","starred_url":"https://api.github.com/users/marekyggdrasil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marekyggdrasil/subscriptions","organizations_url":"https://api.github.com/users/marekyggdrasil/orgs","repos_url":"https://api.github.com/users/marekyggdrasil/repos","events_url":"https://api.github.com/users/marekyggdrasil/events{/privacy}","received_events_url":"https://api.github.com/users/marekyggdrasil/received_events","type":"User","site_admin":false},"assignees":[{"login":"marekyggdrasil","id":8202674,"avatar_url":"https://avatars3.githubusercontent.com/u/8202674?v=3","gravatar_id":"","url":"https://api.github.com/users/marekyggdrasil","html_url":"https://github.com/marekyggdrasil","followers_url":"https://api.github.com/users/marekyggdrasil/followers","following_url":"https://api.github.com/users/marekyggdrasil/following{/other_user}","gists_url":"https://api.github.com/users/marekyggdrasil/gists{/gist_id}","starred_url":"https://api.github.com/users/marekyggdrasil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marekyggdrasil/subscriptions","organizations_url":"https://api.github.com/users/marekyggdrasil/orgs","repos_url":"https://api.github.com/users/marekyggdrasil/repos","events_url":"https://api.github.com/users/marekyggdrasil/events{/privacy}","received_events_url":"https://api.github.com/users/marekyggdrasil/received_events","type":"User","site_admin":false}],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/74/commits","review_comments_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/74/comments","review_comment_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/issues/74/comments","statuses_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/statuses/fe80bb451270148ba9d025da952d072de11da4b7","head":{"label":"marekyggdrasil:feature/67","ref":"feature/67","sha":"fe80bb451270148ba9d025da952d072de11da4b7","user":{"login":"marekyggdrasil","id":8202674,"avatar_url":"https://avatars3.githubusercontent.com/u/8202674?v=3","gravatar_id":"","url":"https://api.github.com/users/marekyggdrasil","html_url":"https://github.com/marekyggdrasil","followers_url":"https://api.github.com/users/marekyggdrasil/followers","following_url":"https://api.github.com/users/marekyggdrasil/following{/other_user}","gists_url":"https://api.github.com/users/marekyggdrasil/gists{/gist_id}","starred_url":"https://api.github.com/users/marekyggdrasil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marekyggdrasil/subscriptions","organizations_url":"https://api.github.com/users/marekyggdrasil/orgs","repos_url":"https://api.github.com/users/marekyggdrasil/repos","events_url":"https://api.github.com/users/marekyggdrasil/events{/privacy}","received_events_url":"https://api.github.com/users/marekyggdrasil/received_events","type":"User","site_admin":false},"repo":{"id":92034246,"name":"beanstalkg","full_name":"marekyggdrasil/beanstalkg","owner":{"login":"marekyggdrasil","id":8202674,"avatar_url":"https://avatars3.githubusercontent.com/u/8202674?v=3","gravatar_id":"","url":"https://api.github.com/users/marekyggdrasil","html_url":"https://github.com/marekyggdrasil","followers_url":"https://api.github.com/users/marekyggdrasil/followers","following_url":"https://api.github.com/users/marekyggdrasil/following{/other_user}","gists_url":"https://api.github.com/users/marekyggdrasil/gists{/gist_id}","starred_url":"https://api.github.com/users/marekyggdrasil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/marekyggdrasil/subscriptions","organizations_url":"https://api.github.com/users/marekyggdrasil/orgs","repos_url":"https://api.github.com/users/marekyggdrasil/repos","events_url":"https://api.github.com/users/marekyggdrasil/events{/privacy}","received_events_url":"https://api.github.com/users/marekyggdrasil/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/marekyggdrasil/beanstalkg","description":"Beanstalkg is a go implementation of beanstalkd - A fast, general-purpose work queue","fork":true,"url":"https://api.github.com/repos/marekyggdrasil/beanstalkg","forks_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/forks","keys_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/keys{/key_id}","collaborators_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/teams","hooks_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/hooks","issue_events_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/issues/events{/number}","events_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/events","assignees_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/assignees{/user}","branches_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/branches{/branch}","tags_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/tags","blobs_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/git/refs{/sha}","trees_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/git/trees{/sha}","statuses_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/statuses/{sha}","languages_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/languages","stargazers_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/stargazers","contributors_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/contributors","subscribers_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/subscribers","subscription_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/subscription","commits_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/commits{/sha}","git_commits_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/git/commits{/sha}","comments_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/comments{/number}","issue_comment_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/issues/comments{/number}","contents_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/contents/{+path}","compare_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/compare/{base}...{head}","merges_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/merges","archive_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/downloads","issues_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/issues{/number}","pulls_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/pulls{/number}","milestones_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/milestones{/number}","notifications_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/labels{/name}","releases_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/releases{/id}","deployments_url":"https://api.github.com/repos/marekyggdrasil/beanstalkg/deployments","created_at":"2017-05-22T09:10:43Z","updated_at":"2017-05-22T09:10:44Z","pushed_at":"2017-06-03T17:54:52Z","git_url":"git://github.com/marekyggdrasil/beanstalkg.git","ssh_url":"git@github.com:marekyggdrasil/beanstalkg.git","clone_url":"https://github.com/marekyggdrasil/beanstalkg.git","svn_url":"https://github.com/marekyggdrasil/beanstalkg","homepage":"","size":317,"stargazers_count":0,"watchers_count":0,"language":"Go","has_issues":false,"has_projects":true,"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":"Beanstalkg:master","ref":"master","sha":"dd0618b99891699953c1cefde0a85afaddb1dccc","user":{"login":"Beanstalkg","id":28643843,"avatar_url":"https://avatars0.githubusercontent.com/u/28643843?v=3","gravatar_id":"","url":"https://api.github.com/users/Beanstalkg","html_url":"https://github.com/Beanstalkg","followers_url":"https://api.github.com/users/Beanstalkg/followers","following_url":"https://api.github.com/users/Beanstalkg/following{/other_user}","gists_url":"https://api.github.com/users/Beanstalkg/gists{/gist_id}","starred_url":"https://api.github.com/users/Beanstalkg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Beanstalkg/subscriptions","organizations_url":"https://api.github.com/users/Beanstalkg/orgs","repos_url":"https://api.github.com/users/Beanstalkg/repos","events_url":"https://api.github.com/users/Beanstalkg/events{/privacy}","received_events_url":"https://api.github.com/users/Beanstalkg/received_events","type":"Organization","site_admin":false},"repo":{"id":74136572,"name":"beanstalkg","full_name":"Beanstalkg/beanstalkg","owner":{"login":"Beanstalkg","id":28643843,"avatar_url":"https://avatars0.githubusercontent.com/u/28643843?v=3","gravatar_id":"","url":"https://api.github.com/users/Beanstalkg","html_url":"https://github.com/Beanstalkg","followers_url":"https://api.github.com/users/Beanstalkg/followers","following_url":"https://api.github.com/users/Beanstalkg/following{/other_user}","gists_url":"https://api.github.com/users/Beanstalkg/gists{/gist_id}","starred_url":"https://api.github.com/users/Beanstalkg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Beanstalkg/subscriptions","organizations_url":"https://api.github.com/users/Beanstalkg/orgs","repos_url":"https://api.github.com/users/Beanstalkg/repos","events_url":"https://api.github.com/users/Beanstalkg/events{/privacy}","received_events_url":"https://api.github.com/users/Beanstalkg/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Beanstalkg/beanstalkg","description":"Beanstalkg is a go implementation of beanstalkd - A fast, general-purpose work queue","fork":false,"url":"https://api.github.com/repos/Beanstalkg/beanstalkg","forks_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/forks","keys_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/teams","hooks_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/hooks","issue_events_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/issues/events{/number}","events_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/events","assignees_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/assignees{/user}","branches_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/branches{/branch}","tags_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/tags","blobs_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/git/refs{/sha}","trees_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/statuses/{sha}","languages_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/languages","stargazers_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/stargazers","contributors_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/contributors","subscribers_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/subscribers","subscription_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/subscription","commits_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/commits{/sha}","git_commits_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/git/commits{/sha}","comments_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/comments{/number}","issue_comment_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/issues/comments{/number}","contents_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/contents/{+path}","compare_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/merges","archive_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/downloads","issues_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/issues{/number}","pulls_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls{/number}","milestones_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/milestones{/number}","notifications_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/labels{/name}","releases_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/releases{/id}","deployments_url":"https://api.github.com/repos/Beanstalkg/beanstalkg/deployments","created_at":"2016-11-18T14:27:08Z","updated_at":"2017-06-05T01:56:22Z","pushed_at":"2017-06-06T11:15:52Z","git_url":"git://github.com/Beanstalkg/beanstalkg.git","ssh_url":"git@github.com:Beanstalkg/beanstalkg.git","clone_url":"https://github.com/Beanstalkg/beanstalkg.git","svn_url":"https://github.com/Beanstalkg/beanstalkg","homepage":"","size":3160,"stargazers_count":9,"watchers_count":9,"language":"Go","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":5,"mirror_url":null,"open_issues_count":23,"forks":5,"open_issues":23,"watchers":9,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/74"},"html":{"href":"https://github.com/Beanstalkg/beanstalkg/pull/74"},"issue":{"href":"https://api.github.com/repos/Beanstalkg/beanstalkg/issues/74"},"comments":{"href":"https://api.github.com/repos/Beanstalkg/beanstalkg/issues/74/comments"},"review_comments":{"href":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/74/comments"},"review_comment":{"href":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Beanstalkg/beanstalkg/pulls/74/commits"},"statuses":{"href":"https://api.github.com/repos/Beanstalkg/beanstalkg/statuses/fe80bb451270148ba9d025da952d072de11da4b7"}}}}
{"actor":{"display_login":"stephenwithav"}}