-
Notifications
You must be signed in to change notification settings - Fork 658
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
chore: move rviz plugins from common to visualization/ folder #9417
Conversation
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
dff9d31
to
652f800
Compare
Signed-off-by: Esteve Fernandez <[email protected]>
652f800
to
9502f9a
Compare
@xmfcx this simply moves the rviz plugins to the visualization folder, there are no code changes. After this PR gets merged, I'll trigger an update of CODEOWNERS file via the corresponding GH action. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9417 +/- ##
==========================================
- Coverage 29.59% 27.93% -1.67%
==========================================
Files 1442 1512 +70
Lines 108512 114960 +6448
Branches 41409 41886 +477
==========================================
Hits 32119 32119
- Misses 73269 79717 +6448
Partials 3124 3124
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Only moves packages 👍
Description
Move
rviz
plugins from comon to its ownvisualization/
folder so that creating Docker images for onlycommon
is easier.Related links
Parent Issue:
autoware.universe/common
packages autoware#5077How was this PR tested?
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.