{"action":"created","issue":{"url":"https://api.github.com/repos/catkin/catkin_tools/issues/163","labels_url":"https://api.github.com/repos/catkin/catkin_tools/issues/163/labels{/name}","comments_url":"https://api.github.com/repos/catkin/catkin_tools/issues/163/comments","events_url":"https://api.github.com/repos/catkin/catkin_tools/issues/163/events","html_url":"https://github.com/catkin/catkin_tools/pull/163","id":62130585,"number":163,"title":"common: suppressing warnings from catkin-pkg when just building","user":{"login":"jbohren","id":1253450,"avatar_url":"https://avatars.githubusercontent.com/u/1253450?v=3","gravatar_id":"","url":"https://api.github.com/users/jbohren","html_url":"https://github.com/jbohren","followers_url":"https://api.github.com/users/jbohren/followers","following_url":"https://api.github.com/users/jbohren/following{/other_user}","gists_url":"https://api.github.com/users/jbohren/gists{/gist_id}","starred_url":"https://api.github.com/users/jbohren/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbohren/subscriptions","organizations_url":"https://api.github.com/users/jbohren/orgs","repos_url":"https://api.github.com/users/jbohren/repos","events_url":"https://api.github.com/users/jbohren/events{/privacy}","received_events_url":"https://api.github.com/users/jbohren/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-03-16T16:17:08Z","updated_at":"2015-04-02T01:45:37Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/catkin/catkin_tools/pulls/163","html_url":"https://github.com/catkin/catkin_tools/pull/163","diff_url":"https://github.com/catkin/catkin_tools/pull/163.diff","patch_url":"https://github.com/catkin/catkin_tools/pull/163.patch"},"body":"This depends on https://github.com/ros-infrastructure/catkin_pkg/pull/114"},"comment":{"url":"https://api.github.com/repos/catkin/catkin_tools/issues/comments/88686299","html_url":"https://github.com/catkin/catkin_tools/pull/163#issuecomment-88686299","issue_url":"https://api.github.com/repos/catkin/catkin_tools/issues/163","id":88686299,"user":{"login":"wjwwood","id":100427,"avatar_url":"https://avatars.githubusercontent.com/u/100427?v=3","gravatar_id":"","url":"https://api.github.com/users/wjwwood","html_url":"https://github.com/wjwwood","followers_url":"https://api.github.com/users/wjwwood/followers","following_url":"https://api.github.com/users/wjwwood/following{/other_user}","gists_url":"https://api.github.com/users/wjwwood/gists{/gist_id}","starred_url":"https://api.github.com/users/wjwwood/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wjwwood/subscriptions","organizations_url":"https://api.github.com/users/wjwwood/orgs","repos_url":"https://api.github.com/users/wjwwood/repos","events_url":"https://api.github.com/users/wjwwood/events{/privacy}","received_events_url":"https://api.github.com/users/wjwwood/received_events","type":"User","site_admin":false},"created_at":"2015-04-02T01:45:37Z","updated_at":"2015-04-02T01:45:37Z","body":"I don't think it is appropriate to suppress the warnings in all the cases, or at least it's not immediately obvious why each of these instances suppresses the warnings.\r\n\r\nCan you expand on why you think it's appropriate to suppress the warnings in each of these cases? If you want to disable them globally then I think it should be a command line configuration and it should not suppress the warnings by default.\r\n\r\nThe warnings are there for a reason and at least in the case of the package naming conventions we want to expose users to the fact their package name doesn't conform as soon as possible because changing the name of a package is harder the longer you wait. If you know what you're doing then you can disable them, but I just don't see the need to suppress them by default."}}