{"action":"created","comment":{"url":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/comments/37315155","id":37315155,"diff_hunk":"@@ -40,7 +40,7 @@ class CostQuery < Report\n             nil\n         end.compact\n \n-        CostQuery.update_all [\"serialized = ?\", YAML::dump(serialized)], [\"id = ?\", query.id]\n+        CostQuery.where([\"id = ?\", query.id]).update_all [\"serialized = ?\", YAML::dump(serialized)]","path":"app/models/cost_query.rb","position":22,"original_position":22,"commit_id":"7d6e4911c6bdcea72322a4cc0f9da8a3d2723e49","original_commit_id":"7d6e4911c6bdcea72322a4cc0f9da8a3d2723e49","user":{"login":"houndci","id":6697940,"avatar_url":"https://avatars.githubusercontent.com/u/6697940?v=3","gravatar_id":"","url":"https://api.github.com/users/houndci","html_url":"https://github.com/houndci","followers_url":"https://api.github.com/users/houndci/followers","following_url":"https://api.github.com/users/houndci/following{/other_user}","gists_url":"https://api.github.com/users/houndci/gists{/gist_id}","starred_url":"https://api.github.com/users/houndci/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/houndci/subscriptions","organizations_url":"https://api.github.com/users/houndci/orgs","repos_url":"https://api.github.com/users/houndci/repos","events_url":"https://api.github.com/users/houndci/events{/privacy}","received_events_url":"https://api.github.com/users/houndci/received_events","type":"User","site_admin":false},"body":"Prefer single-quoted strings when you don't need string interpolation or special symbols.","created_at":"2015-08-18T15:46:58Z","updated_at":"2015-08-18T15:46:58Z","html_url":"https://github.com/finnlabs/openproject-reporting/pull/59#discussion_r37315155","pull_request_url":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/59","_links":{"self":{"href":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/comments/37315155"},"html":{"href":"https://github.com/finnlabs/openproject-reporting/pull/59#discussion_r37315155"},"pull_request":{"href":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/59"}}},"pull_request":{"url":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/59","id":37087316,"html_url":"https://github.com/finnlabs/openproject-reporting/pull/59","diff_url":"https://github.com/finnlabs/openproject-reporting/pull/59.diff","patch_url":"https://github.com/finnlabs/openproject-reporting/pull/59.patch","issue_url":"https://api.github.com/repos/finnlabs/openproject-reporting/issues/59","number":59,"state":"open","locked":false,"title":"[20040] Upgrade to Rails 4.0.x","user":{"login":"myabc","id":755,"avatar_url":"https://avatars.githubusercontent.com/u/755?v=3","gravatar_id":"","url":"https://api.github.com/users/myabc","html_url":"https://github.com/myabc","followers_url":"https://api.github.com/users/myabc/followers","following_url":"https://api.github.com/users/myabc/following{/other_user}","gists_url":"https://api.github.com/users/myabc/gists{/gist_id}","starred_url":"https://api.github.com/users/myabc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/myabc/subscriptions","organizations_url":"https://api.github.com/users/myabc/orgs","repos_url":"https://api.github.com/users/myabc/repos","events_url":"https://api.github.com/users/myabc/events{/privacy}","received_events_url":"https://api.github.com/users/myabc/received_events","type":"User","site_admin":false},"body":":warning: **Please review, merge opf/openproject#2104, finnlabs/reporting_engine#48 first**\r\n\r\nhttps://community.openproject.org/work_packages/20040\r\nhttps://community.openproject.org/work_packages/19916 (Team 2 internal)","created_at":"2015-06-06T08:36:07Z","updated_at":"2015-08-18T15:46:58Z","closed_at":null,"merged_at":null,"merge_commit_sha":"d331d14a435a537d67fcbdffe9faa0eb7dbcf7a7","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/59/commits","review_comments_url":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/59/comments","review_comment_url":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/comments{/number}","comments_url":"https://api.github.com/repos/finnlabs/openproject-reporting/issues/59/comments","statuses_url":"https://api.github.com/repos/finnlabs/openproject-reporting/statuses/7d6e4911c6bdcea72322a4cc0f9da8a3d2723e49","head":{"label":"finnlabs:feature/rails4","ref":"feature/rails4","sha":"7d6e4911c6bdcea72322a4cc0f9da8a3d2723e49","user":{"login":"finnlabs","id":86719,"avatar_url":"https://avatars.githubusercontent.com/u/86719?v=3","gravatar_id":"","url":"https://api.github.com/users/finnlabs","html_url":"https://github.com/finnlabs","followers_url":"https://api.github.com/users/finnlabs/followers","following_url":"https://api.github.com/users/finnlabs/following{/other_user}","gists_url":"https://api.github.com/users/finnlabs/gists{/gist_id}","starred_url":"https://api.github.com/users/finnlabs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/finnlabs/subscriptions","organizations_url":"https://api.github.com/users/finnlabs/orgs","repos_url":"https://api.github.com/users/finnlabs/repos","events_url":"https://api.github.com/users/finnlabs/events{/privacy}","received_events_url":"https://api.github.com/users/finnlabs/received_events","type":"Organization","site_admin":false},"repo":{"id":10028446,"name":"openproject-reporting","full_name":"finnlabs/openproject-reporting","owner":{"login":"finnlabs","id":86719,"avatar_url":"https://avatars.githubusercontent.com/u/86719?v=3","gravatar_id":"","url":"https://api.github.com/users/finnlabs","html_url":"https://github.com/finnlabs","followers_url":"https://api.github.com/users/finnlabs/followers","following_url":"https://api.github.com/users/finnlabs/following{/other_user}","gists_url":"https://api.github.com/users/finnlabs/gists{/gist_id}","starred_url":"https://api.github.com/users/finnlabs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/finnlabs/subscriptions","organizations_url":"https://api.github.com/users/finnlabs/orgs","repos_url":"https://api.github.com/users/finnlabs/repos","events_url":"https://api.github.com/users/finnlabs/events{/privacy}","received_events_url":"https://api.github.com/users/finnlabs/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/finnlabs/openproject-reporting","description":"","fork":false,"url":"https://api.github.com/repos/finnlabs/openproject-reporting","forks_url":"https://api.github.com/repos/finnlabs/openproject-reporting/forks","keys_url":"https://api.github.com/repos/finnlabs/openproject-reporting/keys{/key_id}","collaborators_url":"https://api.github.com/repos/finnlabs/openproject-reporting/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/finnlabs/openproject-reporting/teams","hooks_url":"https://api.github.com/repos/finnlabs/openproject-reporting/hooks","issue_events_url":"https://api.github.com/repos/finnlabs/openproject-reporting/issues/events{/number}","events_url":"https://api.github.com/repos/finnlabs/openproject-reporting/events","assignees_url":"https://api.github.com/repos/finnlabs/openproject-reporting/assignees{/user}","branches_url":"https://api.github.com/repos/finnlabs/openproject-reporting/branches{/branch}","tags_url":"https://api.github.com/repos/finnlabs/openproject-reporting/tags","blobs_url":"https://api.github.com/repos/finnlabs/openproject-reporting/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/finnlabs/openproject-reporting/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/finnlabs/openproject-reporting/git/refs{/sha}","trees_url":"https://api.github.com/repos/finnlabs/openproject-reporting/git/trees{/sha}","statuses_url":"https://api.github.com/repos/finnlabs/openproject-reporting/statuses/{sha}","languages_url":"https://api.github.com/repos/finnlabs/openproject-reporting/languages","stargazers_url":"https://api.github.com/repos/finnlabs/openproject-reporting/stargazers","contributors_url":"https://api.github.com/repos/finnlabs/openproject-reporting/contributors","subscribers_url":"https://api.github.com/repos/finnlabs/openproject-reporting/subscribers","subscription_url":"https://api.github.com/repos/finnlabs/openproject-reporting/subscription","commits_url":"https://api.github.com/repos/finnlabs/openproject-reporting/commits{/sha}","git_commits_url":"https://api.github.com/repos/finnlabs/openproject-reporting/git/commits{/sha}","comments_url":"https://api.github.com/repos/finnlabs/openproject-reporting/comments{/number}","issue_comment_url":"https://api.github.com/repos/finnlabs/openproject-reporting/issues/comments{/number}","contents_url":"https://api.github.com/repos/finnlabs/openproject-reporting/contents/{+path}","compare_url":"https://api.github.com/repos/finnlabs/openproject-reporting/compare/{base}...{head}","merges_url":"https://api.github.com/repos/finnlabs/openproject-reporting/merges","archive_url":"https://api.github.com/repos/finnlabs/openproject-reporting/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/finnlabs/openproject-reporting/downloads","issues_url":"https://api.github.com/repos/finnlabs/openproject-reporting/issues{/number}","pulls_url":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls{/number}","milestones_url":"https://api.github.com/repos/finnlabs/openproject-reporting/milestones{/number}","notifications_url":"https://api.github.com/repos/finnlabs/openproject-reporting/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/finnlabs/openproject-reporting/labels{/name}","releases_url":"https://api.github.com/repos/finnlabs/openproject-reporting/releases{/id}","created_at":"2013-05-13T09:27:44Z","updated_at":"2015-04-10T14:58:05Z","pushed_at":"2015-08-18T15:46:52Z","git_url":"git://github.com/finnlabs/openproject-reporting.git","ssh_url":"git@github.com:finnlabs/openproject-reporting.git","clone_url":"https://github.com/finnlabs/openproject-reporting.git","svn_url":"https://github.com/finnlabs/openproject-reporting","homepage":null,"size":2956,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":10,"mirror_url":null,"open_issues_count":4,"forks":10,"open_issues":4,"watchers":0,"default_branch":"dev"}},"base":{"label":"finnlabs:dev","ref":"dev","sha":"64ca361303d9315cd36b250db8fd4455845baa7c","user":{"login":"finnlabs","id":86719,"avatar_url":"https://avatars.githubusercontent.com/u/86719?v=3","gravatar_id":"","url":"https://api.github.com/users/finnlabs","html_url":"https://github.com/finnlabs","followers_url":"https://api.github.com/users/finnlabs/followers","following_url":"https://api.github.com/users/finnlabs/following{/other_user}","gists_url":"https://api.github.com/users/finnlabs/gists{/gist_id}","starred_url":"https://api.github.com/users/finnlabs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/finnlabs/subscriptions","organizations_url":"https://api.github.com/users/finnlabs/orgs","repos_url":"https://api.github.com/users/finnlabs/repos","events_url":"https://api.github.com/users/finnlabs/events{/privacy}","received_events_url":"https://api.github.com/users/finnlabs/received_events","type":"Organization","site_admin":false},"repo":{"id":10028446,"name":"openproject-reporting","full_name":"finnlabs/openproject-reporting","owner":{"login":"finnlabs","id":86719,"avatar_url":"https://avatars.githubusercontent.com/u/86719?v=3","gravatar_id":"","url":"https://api.github.com/users/finnlabs","html_url":"https://github.com/finnlabs","followers_url":"https://api.github.com/users/finnlabs/followers","following_url":"https://api.github.com/users/finnlabs/following{/other_user}","gists_url":"https://api.github.com/users/finnlabs/gists{/gist_id}","starred_url":"https://api.github.com/users/finnlabs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/finnlabs/subscriptions","organizations_url":"https://api.github.com/users/finnlabs/orgs","repos_url":"https://api.github.com/users/finnlabs/repos","events_url":"https://api.github.com/users/finnlabs/events{/privacy}","received_events_url":"https://api.github.com/users/finnlabs/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/finnlabs/openproject-reporting","description":"","fork":false,"url":"https://api.github.com/repos/finnlabs/openproject-reporting","forks_url":"https://api.github.com/repos/finnlabs/openproject-reporting/forks","keys_url":"https://api.github.com/repos/finnlabs/openproject-reporting/keys{/key_id}","collaborators_url":"https://api.github.com/repos/finnlabs/openproject-reporting/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/finnlabs/openproject-reporting/teams","hooks_url":"https://api.github.com/repos/finnlabs/openproject-reporting/hooks","issue_events_url":"https://api.github.com/repos/finnlabs/openproject-reporting/issues/events{/number}","events_url":"https://api.github.com/repos/finnlabs/openproject-reporting/events","assignees_url":"https://api.github.com/repos/finnlabs/openproject-reporting/assignees{/user}","branches_url":"https://api.github.com/repos/finnlabs/openproject-reporting/branches{/branch}","tags_url":"https://api.github.com/repos/finnlabs/openproject-reporting/tags","blobs_url":"https://api.github.com/repos/finnlabs/openproject-reporting/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/finnlabs/openproject-reporting/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/finnlabs/openproject-reporting/git/refs{/sha}","trees_url":"https://api.github.com/repos/finnlabs/openproject-reporting/git/trees{/sha}","statuses_url":"https://api.github.com/repos/finnlabs/openproject-reporting/statuses/{sha}","languages_url":"https://api.github.com/repos/finnlabs/openproject-reporting/languages","stargazers_url":"https://api.github.com/repos/finnlabs/openproject-reporting/stargazers","contributors_url":"https://api.github.com/repos/finnlabs/openproject-reporting/contributors","subscribers_url":"https://api.github.com/repos/finnlabs/openproject-reporting/subscribers","subscription_url":"https://api.github.com/repos/finnlabs/openproject-reporting/subscription","commits_url":"https://api.github.com/repos/finnlabs/openproject-reporting/commits{/sha}","git_commits_url":"https://api.github.com/repos/finnlabs/openproject-reporting/git/commits{/sha}","comments_url":"https://api.github.com/repos/finnlabs/openproject-reporting/comments{/number}","issue_comment_url":"https://api.github.com/repos/finnlabs/openproject-reporting/issues/comments{/number}","contents_url":"https://api.github.com/repos/finnlabs/openproject-reporting/contents/{+path}","compare_url":"https://api.github.com/repos/finnlabs/openproject-reporting/compare/{base}...{head}","merges_url":"https://api.github.com/repos/finnlabs/openproject-reporting/merges","archive_url":"https://api.github.com/repos/finnlabs/openproject-reporting/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/finnlabs/openproject-reporting/downloads","issues_url":"https://api.github.com/repos/finnlabs/openproject-reporting/issues{/number}","pulls_url":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls{/number}","milestones_url":"https://api.github.com/repos/finnlabs/openproject-reporting/milestones{/number}","notifications_url":"https://api.github.com/repos/finnlabs/openproject-reporting/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/finnlabs/openproject-reporting/labels{/name}","releases_url":"https://api.github.com/repos/finnlabs/openproject-reporting/releases{/id}","created_at":"2013-05-13T09:27:44Z","updated_at":"2015-04-10T14:58:05Z","pushed_at":"2015-08-18T15:46:52Z","git_url":"git://github.com/finnlabs/openproject-reporting.git","ssh_url":"git@github.com:finnlabs/openproject-reporting.git","clone_url":"https://github.com/finnlabs/openproject-reporting.git","svn_url":"https://github.com/finnlabs/openproject-reporting","homepage":null,"size":2956,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":10,"mirror_url":null,"open_issues_count":4,"forks":10,"open_issues":4,"watchers":0,"default_branch":"dev"}},"_links":{"self":{"href":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/59"},"html":{"href":"https://github.com/finnlabs/openproject-reporting/pull/59"},"issue":{"href":"https://api.github.com/repos/finnlabs/openproject-reporting/issues/59"},"comments":{"href":"https://api.github.com/repos/finnlabs/openproject-reporting/issues/59/comments"},"review_comments":{"href":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/59/comments"},"review_comment":{"href":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/finnlabs/openproject-reporting/pulls/59/commits"},"statuses":{"href":"https://api.github.com/repos/finnlabs/openproject-reporting/statuses/7d6e4911c6bdcea72322a4cc0f9da8a3d2723e49"}}}}