-
Notifications
You must be signed in to change notification settings - Fork 673
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(diagnostic_graph_utils): remove unnecessary dependency in diagnostic_graph_utils #9922
base: main
Are you sure you want to change the base?
feat(diagnostic_graph_utils): remove unnecessary dependency in diagnostic_graph_utils #9922
Conversation
…es system/diagnostic_graph_utils Signed-off-by: vish0012 <[email protected]>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9922 +/- ##
==========================================
+ Coverage 29.18% 29.24% +0.06%
==========================================
Files 1410 1413 +3
Lines 107239 107179 -60
Branches 42481 42451 -30
==========================================
+ Hits 31300 31349 +49
+ Misses 72720 72605 -115
- Partials 3219 3225 +6
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
It seems like this most of the modification was already fixed in #9741 |
I have changed the title to just change the package xml to remove the dependency to unused tier4_debug_msgs. |
Description
The tier4_debug_msgs have been replaced with autoware_internal_debug_msgs to enhance clarity and consistency in the codebase.
autowarefoundation/autoware#5580
Related links
autowarefoundation/autoware#5580
Parent Issue:
How was this PR tested?
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.