-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
feat: add autoware_auto_msgs_adapter #4802
feat: add autoware_auto_msgs_adapter #4802
Conversation
I have made this PR a draft so that this won't be merged before autowarefoundation/autoware.universe#7220, but this is ready for review. |
c5766fd
to
cdd9a33
Compare
Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp>
cdd9a33
to
f52ea95
Compare
|
@@ -73,6 +73,10 @@ repositories: | |||
type: git | |||
url: https://github.com/tier4/glog.git | |||
version: v0.6.0_t4-ros | |||
universe/external/autoware_auto_msgs_adapter: # TODO(mitsudome-r): Remove when transition period to autoware_msgs is over |
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.
universe/external/autoware_auto_msgs_adapter: # TODO(mitsudome-r): Remove when transition period to autoware_msgs is over | |
universe/autoware_auto_msgs_adapter: # TODO(mitsudome-r): Remove when transition period to autoware_msgs is over |
External folders were for repositories outside of the https://github.com/autowarefoundation organization.
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.
We should probably add it as a rule somewhere, glog directory is also in external even though it is in this organization.
Since all messages have been converted to autoware_msgs, we don't need it anymore. |
Description
This is part of autowarefoundation/autoware.universe#7219
This must be merged after autowarefoundation/autoware.universe#7220 to avoid duplicate package within the same colcon workspace.
Tests performed
Build test done on local machine.
Effects on system behavior
None
Interface changes
None
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.