-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Add Cone Geometry Message #47
Conversation
859b18d
to
7e2622f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me. Just one minor comment
proto/ignition/msgs/conegeom.proto
Outdated
@@ -0,0 +1,36 @@ | |||
/* | |||
* Copyright (C) 2017 Open Source Robotics Foundation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you update the year to 2020
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
can you sign off your commit? We are now requiring this for DCO check to pass. You can click on Details beside |
9d1c6ed
to
e2ffa57
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the PR.
My only comment is about documentation. We've been trying to at least document the new messages.
e2ffa57
to
ec8f802
Compare
Signed-off-by: Levi Armstrong <levi.armstrong@gmail.com>
ec8f802
to
2d471b7
Compare
No description provided.