{"action":"created","issue":{"url":"https://api.github.com/repos/bbatsov/rubocop/issues/2414","labels_url":"https://api.github.com/repos/bbatsov/rubocop/issues/2414/labels{/name}","comments_url":"https://api.github.com/repos/bbatsov/rubocop/issues/2414/comments","events_url":"https://api.github.com/repos/bbatsov/rubocop/issues/2414/events","html_url":"https://github.com/bbatsov/rubocop/pull/2414","id":116173472,"number":2414,"title":"Call :expression on node instead of :keyword loc","user":{"login":"BlakeWilliams","id":342554,"avatar_url":"https://avatars.githubusercontent.com/u/342554?v=3","gravatar_id":"","url":"https://api.github.com/users/BlakeWilliams","html_url":"https://github.com/BlakeWilliams","followers_url":"https://api.github.com/users/BlakeWilliams/followers","following_url":"https://api.github.com/users/BlakeWilliams/following{/other_user}","gists_url":"https://api.github.com/users/BlakeWilliams/gists{/gist_id}","starred_url":"https://api.github.com/users/BlakeWilliams/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BlakeWilliams/subscriptions","organizations_url":"https://api.github.com/users/BlakeWilliams/orgs","repos_url":"https://api.github.com/users/BlakeWilliams/repos","events_url":"https://api.github.com/users/BlakeWilliams/events{/privacy}","received_events_url":"https://api.github.com/users/BlakeWilliams/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":16,"created_at":"2015-11-10T18:39:38Z","updated_at":"2015-12-20T20:25:14Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/bbatsov/rubocop/pulls/2414","html_url":"https://github.com/bbatsov/rubocop/pull/2414","diff_url":"https://github.com/bbatsov/rubocop/pull/2414.diff","patch_url":"https://github.com/bbatsov/rubocop/pull/2414.patch"},"body":"This changes `RuboCop::Cop::CodeLength`s `check_code_length` method to\r\nreturn `node.loc` instead of `:keyword`.\r\n\r\nWhen `:keyword` is passed in it calls `node.loc.keyword` which returns\r\nthe keyword, eg: `class`, `def`, or `module` and only reports the line\r\nwith that keyword as the beginning and end lines.\r\n\r\nThis passes `:expression` instead which will return the entire block\r\ninstead of the defining line."},"comment":{"url":"https://api.github.com/repos/bbatsov/rubocop/issues/comments/166151584","html_url":"https://github.com/bbatsov/rubocop/pull/2414#issuecomment-166151584","issue_url":"https://api.github.com/repos/bbatsov/rubocop/issues/2414","id":166151584,"user":{"login":"rrosenblum","id":1528852,"avatar_url":"https://avatars.githubusercontent.com/u/1528852?v=3","gravatar_id":"","url":"https://api.github.com/users/rrosenblum","html_url":"https://github.com/rrosenblum","followers_url":"https://api.github.com/users/rrosenblum/followers","following_url":"https://api.github.com/users/rrosenblum/following{/other_user}","gists_url":"https://api.github.com/users/rrosenblum/gists{/gist_id}","starred_url":"https://api.github.com/users/rrosenblum/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rrosenblum/subscriptions","organizations_url":"https://api.github.com/users/rrosenblum/orgs","repos_url":"https://api.github.com/users/rrosenblum/repos","events_url":"https://api.github.com/users/rrosenblum/events{/privacy}","received_events_url":"https://api.github.com/users/rrosenblum/received_events","type":"User","site_admin":false},"created_at":"2015-12-20T20:25:14Z","updated_at":"2015-12-20T20:25:14Z","body":"I would be happy to continue this work. I am not sure what other work needs to be done to this. I know you mentioned adding a dependency so that the change shows necessity. Was there anything else that you wanted done?"}}