{"action":"opened","issue":{"url":"https://api.github.com/repos/checkstyle/contribution/issues/88","repository_url":"https://api.github.com/repos/checkstyle/contribution","labels_url":"https://api.github.com/repos/checkstyle/contribution/issues/88/labels{/name}","comments_url":"https://api.github.com/repos/checkstyle/contribution/issues/88/comments","events_url":"https://api.github.com/repos/checkstyle/contribution/issues/88/events","html_url":"https://github.com/checkstyle/contribution/issues/88","id":151938526,"number":88,"title":"patch-diff-report-tool: report uniques differences","user":{"login":"rnveach","id":5427943,"avatar_url":"https://avatars.githubusercontent.com/u/5427943?v=3","gravatar_id":"","url":"https://api.github.com/users/rnveach","html_url":"https://github.com/rnveach","followers_url":"https://api.github.com/users/rnveach/followers","following_url":"https://api.github.com/users/rnveach/following{/other_user}","gists_url":"https://api.github.com/users/rnveach/gists{/gist_id}","starred_url":"https://api.github.com/users/rnveach/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rnveach/subscriptions","organizations_url":"https://api.github.com/users/rnveach/orgs","repos_url":"https://api.github.com/users/rnveach/repos","events_url":"https://api.github.com/users/rnveach/events{/privacy}","received_events_url":"https://api.github.com/users/rnveach/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-04-29T18:24:46Z","updated_at":"2016-04-29T18:24:46Z","closed_at":null,"body":"Current report total unique violations for base and patch, and how many total unique violations there are.\r\nIt would be nice if we also reported separately the number of new violations in patch and the number of removed violations in base. This is useful if I am expecting no old violations to change and should only see new violations.\r\n\r\nExample:\r\nhttp://rnveach.github.io/checkstyle/2763/Hbase/index.html\r\n````\r\nReport id \tFiles \tUnique rows \tSeverity-warning\r\nbase \t7683 \t14289 \t14289\r\npatch \t2561 \t15201 \t15201\r\ndifference \t272 \t912 \t912\r\n````\r\n\r\nbase Unique rows = unique violations for base = 14289\r\npatch Unique rows = unique violations for patch = 15201\r\ndifference Unique rows = total unique violations = 912\r\n\r\nSince this is a new part of the check, I am expecting all violations to be new, but its not easy to go through 912 manually, and very easy to miss one. Though I will still look at the majority of cases, the summary helps point out possible real issues that should be looked at.\r\n\r\nWhat I am asking for is 2 new counts that would say for this report:\r\n````\r\nOld violations not showing: 0 (count of all orange/base unique rows in report)\r\nNew violations: 912 (count of all green/patch unique rows in report)\r\n````\r\n"}}