{"action":"created","issue":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/7477","repository_url":"https://api.github.com/repos/RobotLocomotion/drake","labels_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/7477/labels{/name}","comments_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/7477/comments","events_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/7477/events","html_url":"https://github.com/RobotLocomotion/drake/pull/7477","id":273984451,"number":7477,"title":"Use clang-format-4.0 on Ubuntu","user":{"login":"jwnimmer-tri","id":17596505,"avatar_url":"https://avatars3.githubusercontent.com/u/17596505?v=4","gravatar_id":"","url":"https://api.github.com/users/jwnimmer-tri","html_url":"https://github.com/jwnimmer-tri","followers_url":"https://api.github.com/users/jwnimmer-tri/followers","following_url":"https://api.github.com/users/jwnimmer-tri/following{/other_user}","gists_url":"https://api.github.com/users/jwnimmer-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/jwnimmer-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jwnimmer-tri/subscriptions","organizations_url":"https://api.github.com/users/jwnimmer-tri/orgs","repos_url":"https://api.github.com/users/jwnimmer-tri/repos","events_url":"https://api.github.com/users/jwnimmer-tri/events{/privacy}","received_events_url":"https://api.github.com/users/jwnimmer-tri/received_events","type":"User","site_admin":false},"labels":[{"id":77798016,"url":"https://api.github.com/repos/RobotLocomotion/drake/labels/priority:%20low","name":"priority: low","color":"e99695","default":false}],"state":"open","locked":false,"assignee":{"login":"ggould-tri","id":17596413,"avatar_url":"https://avatars2.githubusercontent.com/u/17596413?v=4","gravatar_id":"","url":"https://api.github.com/users/ggould-tri","html_url":"https://github.com/ggould-tri","followers_url":"https://api.github.com/users/ggould-tri/followers","following_url":"https://api.github.com/users/ggould-tri/following{/other_user}","gists_url":"https://api.github.com/users/ggould-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/ggould-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ggould-tri/subscriptions","organizations_url":"https://api.github.com/users/ggould-tri/orgs","repos_url":"https://api.github.com/users/ggould-tri/repos","events_url":"https://api.github.com/users/ggould-tri/events{/privacy}","received_events_url":"https://api.github.com/users/ggould-tri/received_events","type":"User","site_admin":false},"assignees":[{"login":"ggould-tri","id":17596413,"avatar_url":"https://avatars2.githubusercontent.com/u/17596413?v=4","gravatar_id":"","url":"https://api.github.com/users/ggould-tri","html_url":"https://github.com/ggould-tri","followers_url":"https://api.github.com/users/ggould-tri/followers","following_url":"https://api.github.com/users/ggould-tri/following{/other_user}","gists_url":"https://api.github.com/users/ggould-tri/gists{/gist_id}","starred_url":"https://api.github.com/users/ggould-tri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ggould-tri/subscriptions","organizations_url":"https://api.github.com/users/ggould-tri/orgs","repos_url":"https://api.github.com/users/ggould-tri/repos","events_url":"https://api.github.com/users/ggould-tri/events{/privacy}","received_events_url":"https://api.github.com/users/ggould-tri/received_events","type":"User","site_admin":false},{"login":"SeanCurtis-TRI","id":22159799,"avatar_url":"https://avatars3.githubusercontent.com/u/22159799?v=4","gravatar_id":"","url":"https://api.github.com/users/SeanCurtis-TRI","html_url":"https://github.com/SeanCurtis-TRI","followers_url":"https://api.github.com/users/SeanCurtis-TRI/followers","following_url":"https://api.github.com/users/SeanCurtis-TRI/following{/other_user}","gists_url":"https://api.github.com/users/SeanCurtis-TRI/gists{/gist_id}","starred_url":"https://api.github.com/users/SeanCurtis-TRI/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SeanCurtis-TRI/subscriptions","organizations_url":"https://api.github.com/users/SeanCurtis-TRI/orgs","repos_url":"https://api.github.com/users/SeanCurtis-TRI/repos","events_url":"https://api.github.com/users/SeanCurtis-TRI/events{/privacy}","received_events_url":"https://api.github.com/users/SeanCurtis-TRI/received_events","type":"User","site_admin":false}],"milestone":null,"comments":4,"created_at":"2017-11-14T23:46:11Z","updated_at":"2017-11-16T15:25:01Z","closed_at":null,"author_association":"COLLABORATOR","pull_request":{"url":"https://api.github.com/repos/RobotLocomotion/drake/pulls/7477","html_url":"https://github.com/RobotLocomotion/drake/pull/7477","diff_url":"https://github.com/RobotLocomotion/drake/pull/7477.diff","patch_url":"https://github.com/RobotLocomotion/drake/pull/7477.patch"},"body":"Update build system linter path to use `clang-format-4.0`.\r\nUpdate documentation to specify `clang-format-4.0`.\r\nRemove `clang-format-3.9` from the install_prereqs list.\r\n\r\nFor reference, the `clang-format-4.0` package was added to `install_prereqs.sh` in #7457.\r\n\r\n<!-- Reviewable:start -->\r\n---\r\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/robotlocomotion/drake/7477)\r\n<!-- Reviewable:end -->\r\n"},"comment":{"url":"https://api.github.com/repos/RobotLocomotion/drake/issues/comments/344956517","html_url":"https://github.com/RobotLocomotion/drake/pull/7477#issuecomment-344956517","issue_url":"https://api.github.com/repos/RobotLocomotion/drake/issues/7477","id":344956517,"user":{"login":"SeanCurtis-TRI","id":22159799,"avatar_url":"https://avatars3.githubusercontent.com/u/22159799?v=4","gravatar_id":"","url":"https://api.github.com/users/SeanCurtis-TRI","html_url":"https://github.com/SeanCurtis-TRI","followers_url":"https://api.github.com/users/SeanCurtis-TRI/followers","following_url":"https://api.github.com/users/SeanCurtis-TRI/following{/other_user}","gists_url":"https://api.github.com/users/SeanCurtis-TRI/gists{/gist_id}","starred_url":"https://api.github.com/users/SeanCurtis-TRI/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SeanCurtis-TRI/subscriptions","organizations_url":"https://api.github.com/users/SeanCurtis-TRI/orgs","repos_url":"https://api.github.com/users/SeanCurtis-TRI/repos","events_url":"https://api.github.com/users/SeanCurtis-TRI/events{/privacy}","received_events_url":"https://api.github.com/users/SeanCurtis-TRI/received_events","type":"User","site_admin":false},"created_at":"2017-11-16T15:25:01Z","updated_at":"2017-11-16T15:25:01Z","author_association":"CONTRIBUTOR","body":"\n\n\n\nReview status: all files reviewed at latest revision, 1 unresolved discussion.\n\n---\n\n*[doc/clion.rst, line 220 at r1](https://reviewable.io:443/reviews/robotlocomotion/drake/7477#-Kz0wZfIKGte9K51XaQ3:-Kz4QRwFEYGRjzFaHB8Q:b18isj5) ([raw file](https://github.com/robotlocomotion/drake/blob/3b8db2fba588b43f7a91c5672d1c2a586a8bf27a/doc/clion.rst#L220)):*\n<details><summary><i>Previously, jwnimmer-tri (Jeremy Nimmer) wrote…</i></summary><blockquote>\n\nMy thought process here was to be minimally disruptive to the existing text and conventions.  If we wanted to go further, I would change `code_style_tools.rst` to remove references to any path symlinking, and instead just spell out `clang-format-4.0` explicitly everywhere in our documentation.  Would that be a useful improvement?\n</blockquote></details>\n\nI think so. So, that would change `:Program: clang-format` to `:Program: clang-format-4.0`. I like that; it's absolutely unambiguous what clang format they should be using and, if advanced users, would rather handle it via links, then that's for them to decide.\n\n---\n\n\n*Comments from [Reviewable](https://reviewable.io:443/reviews/robotlocomotion/drake/7477)*\n<!-- Sent from Reviewable.io -->\n"}}
{"actor":{"display_login":"SeanCurtis-TRI"}}