{"action":"created","comment":{"url":"https://api.github.com/repos/rpappalax/ff-tool/pulls/comments/56099964","id":56099964,"diff_hunk":"@@ -57,15 +48,16 @@ def download(channel):\n \n     download_filename = env.get(channel, 'DOWNLOAD_FILENAME')\n     download_path = os.path.join(BASE_DIR, download_filename)\n+\n+    args = {\"channel\": channel, \"download_path\": download_path}\n+    print(\"Downloading {channel} to {download_path}\".format(**args))\n+","path":"fftool/firefox_download.py","position":23,"original_position":23,"commit_id":"0e9ffe2ef955aa4a134841a428f74aafe4a7690f","original_commit_id":"0e9ffe2ef955aa4a134841a428f74aafe4a7690f","user":{"login":"pdehaan","id":557895,"avatar_url":"https://avatars.githubusercontent.com/u/557895?v=3","gravatar_id":"","url":"https://api.github.com/users/pdehaan","html_url":"https://github.com/pdehaan","followers_url":"https://api.github.com/users/pdehaan/followers","following_url":"https://api.github.com/users/pdehaan/following{/other_user}","gists_url":"https://api.github.com/users/pdehaan/gists{/gist_id}","starred_url":"https://api.github.com/users/pdehaan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pdehaan/subscriptions","organizations_url":"https://api.github.com/users/pdehaan/orgs","repos_url":"https://api.github.com/users/pdehaan/repos","events_url":"https://api.github.com/users/pdehaan/events{/privacy}","received_events_url":"https://api.github.com/users/pdehaan/received_events","type":"User","site_admin":false},"body":"Moved this up, so I could have the `scraper` code a bit closer together below.","created_at":"2016-03-15T00:39:13Z","updated_at":"2016-03-15T00:39:13Z","html_url":"https://github.com/rpappalax/ff-tool/pull/18#discussion_r56099964","pull_request_url":"https://api.github.com/repos/rpappalax/ff-tool/pulls/18","_links":{"self":{"href":"https://api.github.com/repos/rpappalax/ff-tool/pulls/comments/56099964"},"html":{"href":"https://github.com/rpappalax/ff-tool/pull/18#discussion_r56099964"},"pull_request":{"href":"https://api.github.com/repos/rpappalax/ff-tool/pulls/18"}}},"pull_request":{"url":"https://api.github.com/repos/rpappalax/ff-tool/pulls/18","id":62839408,"html_url":"https://github.com/rpappalax/ff-tool/pull/18","diff_url":"https://github.com/rpappalax/ff-tool/pull/18.diff","patch_url":"https://github.com/rpappalax/ff-tool/pull/18.patch","issue_url":"https://api.github.com/repos/rpappalax/ff-tool/issues/18","number":18,"state":"open","locked":false,"title":"Abstract the CLI ArgumentParser code into its own module","user":{"login":"pdehaan","id":557895,"avatar_url":"https://avatars.githubusercontent.com/u/557895?v=3","gravatar_id":"","url":"https://api.github.com/users/pdehaan","html_url":"https://github.com/pdehaan","followers_url":"https://api.github.com/users/pdehaan/followers","following_url":"https://api.github.com/users/pdehaan/following{/other_user}","gists_url":"https://api.github.com/users/pdehaan/gists{/gist_id}","starred_url":"https://api.github.com/users/pdehaan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pdehaan/subscriptions","organizations_url":"https://api.github.com/users/pdehaan/orgs","repos_url":"https://api.github.com/users/pdehaan/repos","events_url":"https://api.github.com/users/pdehaan/events{/privacy}","received_events_url":"https://api.github.com/users/pdehaan/received_events","type":"User","site_admin":false},"body":"r? @rpappalax ","created_at":"2016-03-15T00:37:45Z","updated_at":"2016-03-15T00:39:13Z","closed_at":null,"merged_at":null,"merge_commit_sha":"aec89332965e716ae5e98bc4b8ca1fa61e95786c","assignee":{"login":"rpappalax","id":3699494,"avatar_url":"https://avatars.githubusercontent.com/u/3699494?v=3","gravatar_id":"","url":"https://api.github.com/users/rpappalax","html_url":"https://github.com/rpappalax","followers_url":"https://api.github.com/users/rpappalax/followers","following_url":"https://api.github.com/users/rpappalax/following{/other_user}","gists_url":"https://api.github.com/users/rpappalax/gists{/gist_id}","starred_url":"https://api.github.com/users/rpappalax/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rpappalax/subscriptions","organizations_url":"https://api.github.com/users/rpappalax/orgs","repos_url":"https://api.github.com/users/rpappalax/repos","events_url":"https://api.github.com/users/rpappalax/events{/privacy}","received_events_url":"https://api.github.com/users/rpappalax/received_events","type":"User","site_admin":false},"milestone":null,"commits_url":"https://api.github.com/repos/rpappalax/ff-tool/pulls/18/commits","review_comments_url":"https://api.github.com/repos/rpappalax/ff-tool/pulls/18/comments","review_comment_url":"https://api.github.com/repos/rpappalax/ff-tool/pulls/comments{/number}","comments_url":"https://api.github.com/repos/rpappalax/ff-tool/issues/18/comments","statuses_url":"https://api.github.com/repos/rpappalax/ff-tool/statuses/0e9ffe2ef955aa4a134841a428f74aafe4a7690f","head":{"label":"rpappalax:abstract-cli","ref":"abstract-cli","sha":"0e9ffe2ef955aa4a134841a428f74aafe4a7690f","user":{"login":"rpappalax","id":3699494,"avatar_url":"https://avatars.githubusercontent.com/u/3699494?v=3","gravatar_id":"","url":"https://api.github.com/users/rpappalax","html_url":"https://github.com/rpappalax","followers_url":"https://api.github.com/users/rpappalax/followers","following_url":"https://api.github.com/users/rpappalax/following{/other_user}","gists_url":"https://api.github.com/users/rpappalax/gists{/gist_id}","starred_url":"https://api.github.com/users/rpappalax/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rpappalax/subscriptions","organizations_url":"https://api.github.com/users/rpappalax/orgs","repos_url":"https://api.github.com/users/rpappalax/repos","events_url":"https://api.github.com/users/rpappalax/events{/privacy}","received_events_url":"https://api.github.com/users/rpappalax/received_events","type":"User","site_admin":false},"repo":{"id":52105437,"name":"ff-tool","full_name":"rpappalax/ff-tool","owner":{"login":"rpappalax","id":3699494,"avatar_url":"https://avatars.githubusercontent.com/u/3699494?v=3","gravatar_id":"","url":"https://api.github.com/users/rpappalax","html_url":"https://github.com/rpappalax","followers_url":"https://api.github.com/users/rpappalax/followers","following_url":"https://api.github.com/users/rpappalax/following{/other_user}","gists_url":"https://api.github.com/users/rpappalax/gists{/gist_id}","starred_url":"https://api.github.com/users/rpappalax/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rpappalax/subscriptions","organizations_url":"https://api.github.com/users/rpappalax/orgs","repos_url":"https://api.github.com/users/rpappalax/repos","events_url":"https://api.github.com/users/rpappalax/events{/privacy}","received_events_url":"https://api.github.com/users/rpappalax/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/rpappalax/ff-tool","description":"Python CLI tool for downloading desktop Firefox version, managing profiles and test prefs","fork":false,"url":"https://api.github.com/repos/rpappalax/ff-tool","forks_url":"https://api.github.com/repos/rpappalax/ff-tool/forks","keys_url":"https://api.github.com/repos/rpappalax/ff-tool/keys{/key_id}","collaborators_url":"https://api.github.com/repos/rpappalax/ff-tool/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/rpappalax/ff-tool/teams","hooks_url":"https://api.github.com/repos/rpappalax/ff-tool/hooks","issue_events_url":"https://api.github.com/repos/rpappalax/ff-tool/issues/events{/number}","events_url":"https://api.github.com/repos/rpappalax/ff-tool/events","assignees_url":"https://api.github.com/repos/rpappalax/ff-tool/assignees{/user}","branches_url":"https://api.github.com/repos/rpappalax/ff-tool/branches{/branch}","tags_url":"https://api.github.com/repos/rpappalax/ff-tool/tags","blobs_url":"https://api.github.com/repos/rpappalax/ff-tool/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/rpappalax/ff-tool/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/rpappalax/ff-tool/git/refs{/sha}","trees_url":"https://api.github.com/repos/rpappalax/ff-tool/git/trees{/sha}","statuses_url":"https://api.github.com/repos/rpappalax/ff-tool/statuses/{sha}","languages_url":"https://api.github.com/repos/rpappalax/ff-tool/languages","stargazers_url":"https://api.github.com/repos/rpappalax/ff-tool/stargazers","contributors_url":"https://api.github.com/repos/rpappalax/ff-tool/contributors","subscribers_url":"https://api.github.com/repos/rpappalax/ff-tool/subscribers","subscription_url":"https://api.github.com/repos/rpappalax/ff-tool/subscription","commits_url":"https://api.github.com/repos/rpappalax/ff-tool/commits{/sha}","git_commits_url":"https://api.github.com/repos/rpappalax/ff-tool/git/commits{/sha}","comments_url":"https://api.github.com/repos/rpappalax/ff-tool/comments{/number}","issue_comment_url":"https://api.github.com/repos/rpappalax/ff-tool/issues/comments{/number}","contents_url":"https://api.github.com/repos/rpappalax/ff-tool/contents/{+path}","compare_url":"https://api.github.com/repos/rpappalax/ff-tool/compare/{base}...{head}","merges_url":"https://api.github.com/repos/rpappalax/ff-tool/merges","archive_url":"https://api.github.com/repos/rpappalax/ff-tool/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/rpappalax/ff-tool/downloads","issues_url":"https://api.github.com/repos/rpappalax/ff-tool/issues{/number}","pulls_url":"https://api.github.com/repos/rpappalax/ff-tool/pulls{/number}","milestones_url":"https://api.github.com/repos/rpappalax/ff-tool/milestones{/number}","notifications_url":"https://api.github.com/repos/rpappalax/ff-tool/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/rpappalax/ff-tool/labels{/name}","releases_url":"https://api.github.com/repos/rpappalax/ff-tool/releases{/id}","deployments_url":"https://api.github.com/repos/rpappalax/ff-tool/deployments","created_at":"2016-02-19T17:45:23Z","updated_at":"2016-02-23T00:45:08Z","pushed_at":"2016-03-15T00:37:45Z","git_url":"git://github.com/rpappalax/ff-tool.git","ssh_url":"git@github.com:rpappalax/ff-tool.git","clone_url":"https://github.com/rpappalax/ff-tool.git","svn_url":"https://github.com/rpappalax/ff-tool","homepage":null,"size":54,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":1,"mirror_url":null,"open_issues_count":6,"forks":1,"open_issues":6,"watchers":0,"default_branch":"dev"}},"base":{"label":"rpappalax:dev","ref":"dev","sha":"26bd711430af1a1f675ee2ea02e5aa74e4906010","user":{"login":"rpappalax","id":3699494,"avatar_url":"https://avatars.githubusercontent.com/u/3699494?v=3","gravatar_id":"","url":"https://api.github.com/users/rpappalax","html_url":"https://github.com/rpappalax","followers_url":"https://api.github.com/users/rpappalax/followers","following_url":"https://api.github.com/users/rpappalax/following{/other_user}","gists_url":"https://api.github.com/users/rpappalax/gists{/gist_id}","starred_url":"https://api.github.com/users/rpappalax/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rpappalax/subscriptions","organizations_url":"https://api.github.com/users/rpappalax/orgs","repos_url":"https://api.github.com/users/rpappalax/repos","events_url":"https://api.github.com/users/rpappalax/events{/privacy}","received_events_url":"https://api.github.com/users/rpappalax/received_events","type":"User","site_admin":false},"repo":{"id":52105437,"name":"ff-tool","full_name":"rpappalax/ff-tool","owner":{"login":"rpappalax","id":3699494,"avatar_url":"https://avatars.githubusercontent.com/u/3699494?v=3","gravatar_id":"","url":"https://api.github.com/users/rpappalax","html_url":"https://github.com/rpappalax","followers_url":"https://api.github.com/users/rpappalax/followers","following_url":"https://api.github.com/users/rpappalax/following{/other_user}","gists_url":"https://api.github.com/users/rpappalax/gists{/gist_id}","starred_url":"https://api.github.com/users/rpappalax/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rpappalax/subscriptions","organizations_url":"https://api.github.com/users/rpappalax/orgs","repos_url":"https://api.github.com/users/rpappalax/repos","events_url":"https://api.github.com/users/rpappalax/events{/privacy}","received_events_url":"https://api.github.com/users/rpappalax/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/rpappalax/ff-tool","description":"Python CLI tool for downloading desktop Firefox version, managing profiles and test prefs","fork":false,"url":"https://api.github.com/repos/rpappalax/ff-tool","forks_url":"https://api.github.com/repos/rpappalax/ff-tool/forks","keys_url":"https://api.github.com/repos/rpappalax/ff-tool/keys{/key_id}","collaborators_url":"https://api.github.com/repos/rpappalax/ff-tool/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/rpappalax/ff-tool/teams","hooks_url":"https://api.github.com/repos/rpappalax/ff-tool/hooks","issue_events_url":"https://api.github.com/repos/rpappalax/ff-tool/issues/events{/number}","events_url":"https://api.github.com/repos/rpappalax/ff-tool/events","assignees_url":"https://api.github.com/repos/rpappalax/ff-tool/assignees{/user}","branches_url":"https://api.github.com/repos/rpappalax/ff-tool/branches{/branch}","tags_url":"https://api.github.com/repos/rpappalax/ff-tool/tags","blobs_url":"https://api.github.com/repos/rpappalax/ff-tool/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/rpappalax/ff-tool/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/rpappalax/ff-tool/git/refs{/sha}","trees_url":"https://api.github.com/repos/rpappalax/ff-tool/git/trees{/sha}","statuses_url":"https://api.github.com/repos/rpappalax/ff-tool/statuses/{sha}","languages_url":"https://api.github.com/repos/rpappalax/ff-tool/languages","stargazers_url":"https://api.github.com/repos/rpappalax/ff-tool/stargazers","contributors_url":"https://api.github.com/repos/rpappalax/ff-tool/contributors","subscribers_url":"https://api.github.com/repos/rpappalax/ff-tool/subscribers","subscription_url":"https://api.github.com/repos/rpappalax/ff-tool/subscription","commits_url":"https://api.github.com/repos/rpappalax/ff-tool/commits{/sha}","git_commits_url":"https://api.github.com/repos/rpappalax/ff-tool/git/commits{/sha}","comments_url":"https://api.github.com/repos/rpappalax/ff-tool/comments{/number}","issue_comment_url":"https://api.github.com/repos/rpappalax/ff-tool/issues/comments{/number}","contents_url":"https://api.github.com/repos/rpappalax/ff-tool/contents/{+path}","compare_url":"https://api.github.com/repos/rpappalax/ff-tool/compare/{base}...{head}","merges_url":"https://api.github.com/repos/rpappalax/ff-tool/merges","archive_url":"https://api.github.com/repos/rpappalax/ff-tool/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/rpappalax/ff-tool/downloads","issues_url":"https://api.github.com/repos/rpappalax/ff-tool/issues{/number}","pulls_url":"https://api.github.com/repos/rpappalax/ff-tool/pulls{/number}","milestones_url":"https://api.github.com/repos/rpappalax/ff-tool/milestones{/number}","notifications_url":"https://api.github.com/repos/rpappalax/ff-tool/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/rpappalax/ff-tool/labels{/name}","releases_url":"https://api.github.com/repos/rpappalax/ff-tool/releases{/id}","deployments_url":"https://api.github.com/repos/rpappalax/ff-tool/deployments","created_at":"2016-02-19T17:45:23Z","updated_at":"2016-02-23T00:45:08Z","pushed_at":"2016-03-15T00:37:45Z","git_url":"git://github.com/rpappalax/ff-tool.git","ssh_url":"git@github.com:rpappalax/ff-tool.git","clone_url":"https://github.com/rpappalax/ff-tool.git","svn_url":"https://github.com/rpappalax/ff-tool","homepage":null,"size":54,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":1,"mirror_url":null,"open_issues_count":6,"forks":1,"open_issues":6,"watchers":0,"default_branch":"dev"}},"_links":{"self":{"href":"https://api.github.com/repos/rpappalax/ff-tool/pulls/18"},"html":{"href":"https://github.com/rpappalax/ff-tool/pull/18"},"issue":{"href":"https://api.github.com/repos/rpappalax/ff-tool/issues/18"},"comments":{"href":"https://api.github.com/repos/rpappalax/ff-tool/issues/18/comments"},"review_comments":{"href":"https://api.github.com/repos/rpappalax/ff-tool/pulls/18/comments"},"review_comment":{"href":"https://api.github.com/repos/rpappalax/ff-tool/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/rpappalax/ff-tool/pulls/18/commits"},"statuses":{"href":"https://api.github.com/repos/rpappalax/ff-tool/statuses/0e9ffe2ef955aa4a134841a428f74aafe4a7690f"}}}}