-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename drake/examples/Cars to drake/automotive #3436
Rename drake/examples/Cars to drake/automotive #3436
Conversation
+@rpoyner-tri for feature review, please. |
This PR moves the files and updates all path references. A subsequent PR will deal with updating the C++ namespaces and any other loose ends. |
+@david-german-tri for platform review, please. |
@drake-jenkins-bot linux-clang-experimental-ros please Review status: 0 of 78 files reviewed at latest revision, all discussions resolved. Comments from Reviewable |
Reviewed 148 of 148 files at r1. Comments from Reviewable |
Ah, just saw your comment above about further loose ends, such as "drakeCars" and "drake_cars" in various places. Reviewed 148 of 148 files at r1. Comments from Reviewable |
Ah bummer, of course the ROS build is going to fail. I always forget that. |
d2f6efe
to
8811cec
Compare
@drake-jenkins-bot linux-clang-experimental-ros please |
Reviewed 4 of 4 files at r2. Comments from Reviewable |
Wow. Three hours from starting typing to a merge. Possibly a new personal best. |
Closes #3406.
This change is