{"action":"created","issue":{"url":"https://api.github.com/repos/apache/groovy/issues/282","repository_url":"https://api.github.com/repos/apache/groovy","labels_url":"https://api.github.com/repos/apache/groovy/issues/282/labels{/name}","comments_url":"https://api.github.com/repos/apache/groovy/issues/282/comments","events_url":"https://api.github.com/repos/apache/groovy/issues/282/events","html_url":"https://github.com/apache/groovy/pull/282","id":138799794,"number":282,"title":"Some bulk refactorings","user":{"login":"oreissig","id":4244988,"avatar_url":"https://avatars.githubusercontent.com/u/4244988?v=3","gravatar_id":"","url":"https://api.github.com/users/oreissig","html_url":"https://github.com/oreissig","followers_url":"https://api.github.com/users/oreissig/followers","following_url":"https://api.github.com/users/oreissig/following{/other_user}","gists_url":"https://api.github.com/users/oreissig/gists{/gist_id}","starred_url":"https://api.github.com/users/oreissig/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oreissig/subscriptions","organizations_url":"https://api.github.com/users/oreissig/orgs","repos_url":"https://api.github.com/users/oreissig/repos","events_url":"https://api.github.com/users/oreissig/events{/privacy}","received_events_url":"https://api.github.com/users/oreissig/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2016-03-06T15:09:14Z","updated_at":"2016-03-10T19:44:14Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/groovy/pulls/282","html_url":"https://github.com/apache/groovy/pull/282","diff_url":"https://github.com/apache/groovy/pull/282.diff","patch_url":"https://github.com/apache/groovy/pull/282.patch"},"body":"I went through the codebase of the root project and did the following refactorings:\r\n- make private methods static when they are plain functions\r\n- add @Deprecated when javadoc says the method is @deprecated\r\n- remove unused code\r\n- avoid synthetic accessors by making private fields/methods package-private\r\n- make private fields final where possible\r\n- fix some inconsistent whitespaces\r\n\r\nNone of this should change the behaviour in any way."},"comment":{"url":"https://api.github.com/repos/apache/groovy/issues/comments/195015648","html_url":"https://github.com/apache/groovy/pull/282#issuecomment-195015648","issue_url":"https://api.github.com/repos/apache/groovy/issues/282","id":195015648,"user":{"login":"oreissig","id":4244988,"avatar_url":"https://avatars.githubusercontent.com/u/4244988?v=3","gravatar_id":"","url":"https://api.github.com/users/oreissig","html_url":"https://github.com/oreissig","followers_url":"https://api.github.com/users/oreissig/followers","following_url":"https://api.github.com/users/oreissig/following{/other_user}","gists_url":"https://api.github.com/users/oreissig/gists{/gist_id}","starred_url":"https://api.github.com/users/oreissig/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/oreissig/subscriptions","organizations_url":"https://api.github.com/users/oreissig/orgs","repos_url":"https://api.github.com/users/oreissig/repos","events_url":"https://api.github.com/users/oreissig/events{/privacy}","received_events_url":"https://api.github.com/users/oreissig/received_events","type":"User","site_admin":false},"created_at":"2016-03-10T19:44:14Z","updated_at":"2016-03-10T19:44:14Z","body":"If you want I can go through again and restructure the changes. How do you like it? Individual PRs for each of the commits (except the fixup)? Should those individual PRs only contain one commit each, or do you like it to be split up by some metric?"}}