{"action":"created","comment":{"url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/comments/25342145","id":25342145,"diff_hunk":"@@ -16,9 +16,12 @@ function isUnsatisfied(pkg) {\n function EmberCLIDependencyChecker(project) {\n   this.name    = 'ember-cli-dependency-checker';\n   this.project = project;\n-  this.checkDependencies();\n }\n \n+EmberCLIDependencyChecker.prototype.included = function(/*app*/) {\n+  this.checkDependencies();","path":"lib/dependency-checker.js","position":8,"original_position":8,"commit_id":"9845638a8b986762d9da414c676a9f407a192929","original_commit_id":"9845638a8b986762d9da414c676a9f407a192929","user":{"login":"stefanpenner","id":1377,"avatar_url":"https://avatars.githubusercontent.com/u/1377?v=3","gravatar_id":"","url":"https://api.github.com/users/stefanpenner","html_url":"https://github.com/stefanpenner","followers_url":"https://api.github.com/users/stefanpenner/followers","following_url":"https://api.github.com/users/stefanpenner/following{/other_user}","gists_url":"https://api.github.com/users/stefanpenner/gists{/gist_id}","starred_url":"https://api.github.com/users/stefanpenner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stefanpenner/subscriptions","organizations_url":"https://api.github.com/users/stefanpenner/orgs","repos_url":"https://api.github.com/users/stefanpenner/repos","events_url":"https://api.github.com/users/stefanpenner/events{/privacy}","received_events_url":"https://api.github.com/users/stefanpenner/received_events","type":"User","site_admin":false},"body":"the included hook currently gets called twice, do to a quirk that isn't fixed yet. We may want to guard.\r\n\r\ncc @rwjblue should confirm","created_at":"2015-02-25T14:19:02Z","updated_at":"2015-02-25T14:19:02Z","html_url":"https://github.com/quaertym/ember-cli-dependency-checker/pull/23#discussion_r25342145","pull_request_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/23","_links":{"self":{"href":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/comments/25342145"},"html":{"href":"https://github.com/quaertym/ember-cli-dependency-checker/pull/23#discussion_r25342145"},"pull_request":{"href":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/23"}}},"pull_request":{"url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/23","id":29988529,"html_url":"https://github.com/quaertym/ember-cli-dependency-checker/pull/23","diff_url":"https://github.com/quaertym/ember-cli-dependency-checker/pull/23.diff","patch_url":"https://github.com/quaertym/ember-cli-dependency-checker/pull/23.patch","issue_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/issues/23","number":23,"state":"open","locked":false,"title":"Move dependency checking to included hook","user":{"login":"quaertym","id":904731,"avatar_url":"https://avatars.githubusercontent.com/u/904731?v=3","gravatar_id":"","url":"https://api.github.com/users/quaertym","html_url":"https://github.com/quaertym","followers_url":"https://api.github.com/users/quaertym/followers","following_url":"https://api.github.com/users/quaertym/following{/other_user}","gists_url":"https://api.github.com/users/quaertym/gists{/gist_id}","starred_url":"https://api.github.com/users/quaertym/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quaertym/subscriptions","organizations_url":"https://api.github.com/users/quaertym/orgs","repos_url":"https://api.github.com/users/quaertym/repos","events_url":"https://api.github.com/users/quaertym/events{/privacy}","received_events_url":"https://api.github.com/users/quaertym/received_events","type":"User","site_admin":false},"body":"","created_at":"2015-02-25T13:58:39Z","updated_at":"2015-02-25T14:19:02Z","closed_at":null,"merged_at":null,"merge_commit_sha":"75a06d7540ff34cd403bfdb0b8988de7a5b0dbef","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/23/commits","review_comments_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/23/comments","review_comment_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/comments{/number}","comments_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/issues/23/comments","statuses_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/statuses/9845638a8b986762d9da414c676a9f407a192929","head":{"label":"quaertym:check-dependencies-when-included","ref":"check-dependencies-when-included","sha":"9845638a8b986762d9da414c676a9f407a192929","user":{"login":"quaertym","id":904731,"avatar_url":"https://avatars.githubusercontent.com/u/904731?v=3","gravatar_id":"","url":"https://api.github.com/users/quaertym","html_url":"https://github.com/quaertym","followers_url":"https://api.github.com/users/quaertym/followers","following_url":"https://api.github.com/users/quaertym/following{/other_user}","gists_url":"https://api.github.com/users/quaertym/gists{/gist_id}","starred_url":"https://api.github.com/users/quaertym/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quaertym/subscriptions","organizations_url":"https://api.github.com/users/quaertym/orgs","repos_url":"https://api.github.com/users/quaertym/repos","events_url":"https://api.github.com/users/quaertym/events{/privacy}","received_events_url":"https://api.github.com/users/quaertym/received_events","type":"User","site_admin":false},"repo":{"id":24802644,"name":"ember-cli-dependency-checker","full_name":"quaertym/ember-cli-dependency-checker","owner":{"login":"quaertym","id":904731,"avatar_url":"https://avatars.githubusercontent.com/u/904731?v=3","gravatar_id":"","url":"https://api.github.com/users/quaertym","html_url":"https://github.com/quaertym","followers_url":"https://api.github.com/users/quaertym/followers","following_url":"https://api.github.com/users/quaertym/following{/other_user}","gists_url":"https://api.github.com/users/quaertym/gists{/gist_id}","starred_url":"https://api.github.com/users/quaertym/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quaertym/subscriptions","organizations_url":"https://api.github.com/users/quaertym/orgs","repos_url":"https://api.github.com/users/quaertym/repos","events_url":"https://api.github.com/users/quaertym/events{/privacy}","received_events_url":"https://api.github.com/users/quaertym/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/quaertym/ember-cli-dependency-checker","description":"Ember CLI addon for checking missing node and bower dependencies before running ember commands","fork":false,"url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker","forks_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/forks","keys_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/keys{/key_id}","collaborators_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/teams","hooks_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/hooks","issue_events_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/issues/events{/number}","events_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/events","assignees_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/assignees{/user}","branches_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/branches{/branch}","tags_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/tags","blobs_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/git/refs{/sha}","trees_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/git/trees{/sha}","statuses_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/statuses/{sha}","languages_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/languages","stargazers_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/stargazers","contributors_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/contributors","subscribers_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/subscribers","subscription_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/subscription","commits_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/commits{/sha}","git_commits_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/git/commits{/sha}","comments_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/comments{/number}","issue_comment_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/issues/comments{/number}","contents_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/contents/{+path}","compare_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/compare/{base}...{head}","merges_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/merges","archive_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/downloads","issues_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/issues{/number}","pulls_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls{/number}","milestones_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/milestones{/number}","notifications_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/labels{/name}","releases_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/releases{/id}","created_at":"2014-10-04T21:54:30Z","updated_at":"2015-02-25T10:54:22Z","pushed_at":"2015-02-25T13:46:58Z","git_url":"git://github.com/quaertym/ember-cli-dependency-checker.git","ssh_url":"git@github.com:quaertym/ember-cli-dependency-checker.git","clone_url":"https://github.com/quaertym/ember-cli-dependency-checker.git","svn_url":"https://github.com/quaertym/ember-cli-dependency-checker","homepage":null,"size":288,"stargazers_count":5,"watchers_count":5,"language":"JavaScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":5,"mirror_url":null,"open_issues_count":5,"forks":5,"open_issues":5,"watchers":5,"default_branch":"master"}},"base":{"label":"quaertym:master","ref":"master","sha":"96b9b2a3599a013b152cff6fb547272fc634f509","user":{"login":"quaertym","id":904731,"avatar_url":"https://avatars.githubusercontent.com/u/904731?v=3","gravatar_id":"","url":"https://api.github.com/users/quaertym","html_url":"https://github.com/quaertym","followers_url":"https://api.github.com/users/quaertym/followers","following_url":"https://api.github.com/users/quaertym/following{/other_user}","gists_url":"https://api.github.com/users/quaertym/gists{/gist_id}","starred_url":"https://api.github.com/users/quaertym/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quaertym/subscriptions","organizations_url":"https://api.github.com/users/quaertym/orgs","repos_url":"https://api.github.com/users/quaertym/repos","events_url":"https://api.github.com/users/quaertym/events{/privacy}","received_events_url":"https://api.github.com/users/quaertym/received_events","type":"User","site_admin":false},"repo":{"id":24802644,"name":"ember-cli-dependency-checker","full_name":"quaertym/ember-cli-dependency-checker","owner":{"login":"quaertym","id":904731,"avatar_url":"https://avatars.githubusercontent.com/u/904731?v=3","gravatar_id":"","url":"https://api.github.com/users/quaertym","html_url":"https://github.com/quaertym","followers_url":"https://api.github.com/users/quaertym/followers","following_url":"https://api.github.com/users/quaertym/following{/other_user}","gists_url":"https://api.github.com/users/quaertym/gists{/gist_id}","starred_url":"https://api.github.com/users/quaertym/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/quaertym/subscriptions","organizations_url":"https://api.github.com/users/quaertym/orgs","repos_url":"https://api.github.com/users/quaertym/repos","events_url":"https://api.github.com/users/quaertym/events{/privacy}","received_events_url":"https://api.github.com/users/quaertym/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/quaertym/ember-cli-dependency-checker","description":"Ember CLI addon for checking missing node and bower dependencies before running ember commands","fork":false,"url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker","forks_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/forks","keys_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/keys{/key_id}","collaborators_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/teams","hooks_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/hooks","issue_events_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/issues/events{/number}","events_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/events","assignees_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/assignees{/user}","branches_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/branches{/branch}","tags_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/tags","blobs_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/git/refs{/sha}","trees_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/git/trees{/sha}","statuses_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/statuses/{sha}","languages_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/languages","stargazers_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/stargazers","contributors_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/contributors","subscribers_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/subscribers","subscription_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/subscription","commits_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/commits{/sha}","git_commits_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/git/commits{/sha}","comments_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/comments{/number}","issue_comment_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/issues/comments{/number}","contents_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/contents/{+path}","compare_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/compare/{base}...{head}","merges_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/merges","archive_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/downloads","issues_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/issues{/number}","pulls_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls{/number}","milestones_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/milestones{/number}","notifications_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/labels{/name}","releases_url":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/releases{/id}","created_at":"2014-10-04T21:54:30Z","updated_at":"2015-02-25T10:54:22Z","pushed_at":"2015-02-25T13:46:58Z","git_url":"git://github.com/quaertym/ember-cli-dependency-checker.git","ssh_url":"git@github.com:quaertym/ember-cli-dependency-checker.git","clone_url":"https://github.com/quaertym/ember-cli-dependency-checker.git","svn_url":"https://github.com/quaertym/ember-cli-dependency-checker","homepage":null,"size":288,"stargazers_count":5,"watchers_count":5,"language":"JavaScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":5,"mirror_url":null,"open_issues_count":5,"forks":5,"open_issues":5,"watchers":5,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/23"},"html":{"href":"https://github.com/quaertym/ember-cli-dependency-checker/pull/23"},"issue":{"href":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/issues/23"},"comments":{"href":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/issues/23/comments"},"review_comments":{"href":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/23/comments"},"review_comment":{"href":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/pulls/23/commits"},"statuses":{"href":"https://api.github.com/repos/quaertym/ember-cli-dependency-checker/statuses/9845638a8b986762d9da414c676a9f407a192929"}}}}