{"comment":{"url":"https://api.github.com/repos/jenkinsci/warnings-plugin/comments/11559231","html_url":"https://github.com/jenkinsci/warnings-plugin/commit/217cbade24ecf62399b2789531f6f8060cb358a1#commitcomment-11559231","id":11559231,"user":{"login":"ianhinder","id":213274,"avatar_url":"https://avatars.githubusercontent.com/u/213274?v=3","gravatar_id":"","url":"https://api.github.com/users/ianhinder","html_url":"https://github.com/ianhinder","followers_url":"https://api.github.com/users/ianhinder/followers","following_url":"https://api.github.com/users/ianhinder/following{/other_user}","gists_url":"https://api.github.com/users/ianhinder/gists{/gist_id}","starred_url":"https://api.github.com/users/ianhinder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ianhinder/subscriptions","organizations_url":"https://api.github.com/users/ianhinder/orgs","repos_url":"https://api.github.com/users/ianhinder/repos","events_url":"https://api.github.com/users/ianhinder/events{/privacy}","received_events_url":"https://api.github.com/users/ianhinder/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"217cbade24ecf62399b2789531f6f8060cb358a1","created_at":"2015-06-08T00:14:08Z","updated_at":"2015-06-08T00:14:08Z","body":"According to http://docs.pylint.org/output.html, the PyLint classifications are:\r\n\r\n    [R]efactor for a “good practice” metric violation\r\n    [C]onvention for coding standard violation\r\n    [W]arning for stylistic problems, or minor programming issues\r\n    [E]rror for important programming issues (i.e. most probably bug)\r\n    [F]atal for errors which prevented further processing\r\n\r\nThe code in this commit reports warnings as high priority; this includes \"TODO:\" items.  This means that \"most probably bug\"s are being grouped together with stylistic problems.  Also, low priority seems to be unused.  I suggest changing the mapping to:\r\n\r\n    Low: R, C\r\n    Medium: W\r\n    High: E, F\r\n\r\nOf course, the best would be if the user could configure this, but that is a larger change.\r\n"}}