{"action":"created","issue":{"url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/600","repository_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api","labels_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/600/labels{/name}","comments_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/600/comments","events_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/600/events","html_url":"https://github.com/eclipse-ee4j/jaxrs-api/pull/600","id":306142577,"number":600,"title":"Adjusted Checkstyle's WhitespaceAround rule for generics (#587)","user":{"login":"chkal","id":159245,"avatar_url":"https://avatars2.githubusercontent.com/u/159245?v=4","gravatar_id":"","url":"https://api.github.com/users/chkal","html_url":"https://github.com/chkal","followers_url":"https://api.github.com/users/chkal/followers","following_url":"https://api.github.com/users/chkal/following{/other_user}","gists_url":"https://api.github.com/users/chkal/gists{/gist_id}","starred_url":"https://api.github.com/users/chkal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chkal/subscriptions","organizations_url":"https://api.github.com/users/chkal/orgs","repos_url":"https://api.github.com/users/chkal/repos","events_url":"https://api.github.com/users/chkal/events{/privacy}","received_events_url":"https://api.github.com/users/chkal/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":{"login":"mkarg","id":1701815,"avatar_url":"https://avatars2.githubusercontent.com/u/1701815?v=4","gravatar_id":"","url":"https://api.github.com/users/mkarg","html_url":"https://github.com/mkarg","followers_url":"https://api.github.com/users/mkarg/followers","following_url":"https://api.github.com/users/mkarg/following{/other_user}","gists_url":"https://api.github.com/users/mkarg/gists{/gist_id}","starred_url":"https://api.github.com/users/mkarg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mkarg/subscriptions","organizations_url":"https://api.github.com/users/mkarg/orgs","repos_url":"https://api.github.com/users/mkarg/repos","events_url":"https://api.github.com/users/mkarg/events{/privacy}","received_events_url":"https://api.github.com/users/mkarg/received_events","type":"User","site_admin":false},"assignees":[{"login":"mkarg","id":1701815,"avatar_url":"https://avatars2.githubusercontent.com/u/1701815?v=4","gravatar_id":"","url":"https://api.github.com/users/mkarg","html_url":"https://github.com/mkarg","followers_url":"https://api.github.com/users/mkarg/followers","following_url":"https://api.github.com/users/mkarg/following{/other_user}","gists_url":"https://api.github.com/users/mkarg/gists{/gist_id}","starred_url":"https://api.github.com/users/mkarg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mkarg/subscriptions","organizations_url":"https://api.github.com/users/mkarg/orgs","repos_url":"https://api.github.com/users/mkarg/repos","events_url":"https://api.github.com/users/mkarg/events{/privacy}","received_events_url":"https://api.github.com/users/mkarg/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2018-03-17T09:33:16Z","updated_at":"2018-03-19T18:28:44Z","closed_at":"2018-03-17T12:53:41Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/pulls/600","html_url":"https://github.com/eclipse-ee4j/jaxrs-api/pull/600","diff_url":"https://github.com/eclipse-ee4j/jaxrs-api/pull/600.diff","patch_url":"https://github.com/eclipse-ee4j/jaxrs-api/pull/600.patch"},"body":"Currently Checkstyle reports 2516 (!) violations in the source code. The number increased dramatically after updating Checkstyle as part of #595.\r\n\r\nA majority of the violations is caused by the `WhitespaceAround` rule, which is currently configured to enforce whitespaces around `<` and `>`. Therefore, this rule is violated for all generic declarations like `List<String>`.\r\n\r\nThis pull requests updates the Checkstyle configuration to not check for whitespaces around `<` and `>` anymore, which reduces the total number of violations to 674."},"comment":{"url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/comments/374317619","html_url":"https://github.com/eclipse-ee4j/jaxrs-api/pull/600#issuecomment-374317619","issue_url":"https://api.github.com/repos/eclipse-ee4j/jaxrs-api/issues/600","id":374317619,"user":{"login":"mkarg","id":1701815,"avatar_url":"https://avatars2.githubusercontent.com/u/1701815?v=4","gravatar_id":"","url":"https://api.github.com/users/mkarg","html_url":"https://github.com/mkarg","followers_url":"https://api.github.com/users/mkarg/followers","following_url":"https://api.github.com/users/mkarg/following{/other_user}","gists_url":"https://api.github.com/users/mkarg/gists{/gist_id}","starred_url":"https://api.github.com/users/mkarg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mkarg/subscriptions","organizations_url":"https://api.github.com/users/mkarg/orgs","repos_url":"https://api.github.com/users/mkarg/repos","events_url":"https://api.github.com/users/mkarg/events{/privacy}","received_events_url":"https://api.github.com/users/mkarg/received_events","type":"User","site_admin":false},"created_at":"2018-03-19T18:28:44Z","updated_at":"2018-03-19T18:28:44Z","author_association":"MEMBER","body":"I personally cannot tell any reason against tweaking the ruleset. In the end, I don't know who set the ruleset up and what his intention was. I would just not remove rules that we actually want to have. For example, FinalParameters will be actually solved by a PR of mine which I am preparing at the moment, so there is no need to remove that rule."}}
{"actor":{"display_login":"mkarg"}}