{"action":"created","issue":{"url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2666","labels_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2666/labels{/name}","comments_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2666/comments","events_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2666/events","html_url":"https://github.com/checkstyle/checkstyle/issues/2666","id":120555201,"number":2666,"title":"Print name of the Check after printing violation message","user":{"login":"romani","id":812984,"avatar_url":"https://avatars.githubusercontent.com/u/812984?v=3","gravatar_id":"","url":"https://api.github.com/users/romani","html_url":"https://github.com/romani","followers_url":"https://api.github.com/users/romani/followers","following_url":"https://api.github.com/users/romani/following{/other_user}","gists_url":"https://api.github.com/users/romani/gists{/gist_id}","starred_url":"https://api.github.com/users/romani/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/romani/subscriptions","organizations_url":"https://api.github.com/users/romani/orgs","repos_url":"https://api.github.com/users/romani/repos","events_url":"https://api.github.com/users/romani/events{/privacy}","received_events_url":"https://api.github.com/users/romani/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/checkstyle/checkstyle/labels/approved","name":"approved","color":"009800"}],"state":"open","locked":false,"assignee":{"login":"MEZk","id":7242568,"avatar_url":"https://avatars.githubusercontent.com/u/7242568?v=3","gravatar_id":"","url":"https://api.github.com/users/MEZk","html_url":"https://github.com/MEZk","followers_url":"https://api.github.com/users/MEZk/followers","following_url":"https://api.github.com/users/MEZk/following{/other_user}","gists_url":"https://api.github.com/users/MEZk/gists{/gist_id}","starred_url":"https://api.github.com/users/MEZk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MEZk/subscriptions","organizations_url":"https://api.github.com/users/MEZk/orgs","repos_url":"https://api.github.com/users/MEZk/repos","events_url":"https://api.github.com/users/MEZk/events{/privacy}","received_events_url":"https://api.github.com/users/MEZk/received_events","type":"User","site_admin":false},"milestone":null,"comments":11,"created_at":"2015-12-05T13:44:21Z","updated_at":"2015-12-07T21:47:00Z","closed_at":null,"body":"right now we have format like:\r\n```\r\n~/Test1.java:5: warning: Uncommented main method found.\r\n```\r\n\r\nit is not clear what Module/Check producing this, it is more actual to naming Checks where format of message is the same for several Check. User do not clearly see what Check config should be adjusted and what Check to search in web documentation , ..... . \r\n\r\nPMD has format:\r\nPMD Failure: com/puppycrawl/tools/checkstyle/api/JavadocTokenTypes.java:31 Rule:ExcessiveClassLength Priority:3 Avoid really long classes..\r\n\r\nFindbugs format:\r\ncom.puppycrawl.tools.checkstyle.checks.UniquePropertiesCheck$UniqueProperties doesn't override java.util.Hashtable.equals(Object) [com.puppycrawl.tools.checkstyle.checks.UniquePropertiesCheck$UniqueProperties] At UniquePropertiesCheck.java:[line 1] EQ_DOESNT_OVERRIDE_EQUALS\r\n\r\n\r\nshould be like (one of) (I am in favour of first or second ):\r\n```\r\n~/Test1.java:5: Check:UncommentedMain warning: Uncommented main method found.\r\n~/Test1.java:5: warning: UncommentedMain: Uncommented main method found.\r\n\r\n~/Test1.java:5: warning: [UncommentedMain] Uncommented main method found.\r\n~/Test1.java:5: warning: Uncommented main method found. [UncommentedMain]\r\n```"},"comment":{"url":"https://api.github.com/repos/checkstyle/checkstyle/issues/comments/162674683","html_url":"https://github.com/checkstyle/checkstyle/issues/2666#issuecomment-162674683","issue_url":"https://api.github.com/repos/checkstyle/checkstyle/issues/2666","id":162674683,"user":{"login":"mkordas","id":5467276,"avatar_url":"https://avatars.githubusercontent.com/u/5467276?v=3","gravatar_id":"","url":"https://api.github.com/users/mkordas","html_url":"https://github.com/mkordas","followers_url":"https://api.github.com/users/mkordas/followers","following_url":"https://api.github.com/users/mkordas/following{/other_user}","gists_url":"https://api.github.com/users/mkordas/gists{/gist_id}","starred_url":"https://api.github.com/users/mkordas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mkordas/subscriptions","organizations_url":"https://api.github.com/users/mkordas/orgs","repos_url":"https://api.github.com/users/mkordas/repos","events_url":"https://api.github.com/users/mkordas/events{/privacy}","received_events_url":"https://api.github.com/users/mkordas/received_events","type":"User","site_admin":false},"created_at":"2015-12-07T21:47:00Z","updated_at":"2015-12-07T21:47:00Z","body":"@romani, I'm OK with your latest comments. However, I have one more proposition, similar to maven-checkstyle-plugin one:\r\n```\r\n[ERROR] My.java:127:5 LeftCurly: '{' at column 5 should be on the previous line.\r\n```\r\nWarning level is first, so it's consistent with logging frameworks. There is no strange double colon as in current format (~/Test1.java:5`:` warning`:`). Maybe message is a bit more important that check name, but on the other hand sometimes messages are pretty long and check names may be too far away."}}