Skip to content
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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

vish0012
Copy link
Contributor

@vish0012 vish0012 commented Jan 15, 2025

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:

  • Link

How was this PR tested?

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

…es system/diagnostic_graph_utils

Signed-off-by: vish0012 <[email protected]>
@github-actions github-actions bot added the component:system System design and integration. (auto-assigned) label Jan 15, 2025
Copy link

github-actions bot commented Jan 15, 2025

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@mitsudome-r mitsudome-r added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Jan 17, 2025
@mitsudome-r mitsudome-r changed the title feat: tier4_debug_msgs changed to autoware_internal_debug_msgs in fil… feat(diagnostic_graph_utils)!: tier4_debug_msgs changed to autoware_internal_debug_msgs in diagnostic_graph_utils Jan 17, 2025
Copy link

codecov bot commented Jan 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 29.24%. Comparing base (b7b3dd6) to head (a5442d9).
Report is 7 commits behind head on main.

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     
Flag Coverage Δ *Carryforward flag
differential 7.68% <ø> (?)
total 29.21% <ø> (+0.02%) ⬆️ Carriedforward from b7b3dd6

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mitsudome-r
Copy link
Member

It seems like this most of the modification was already fixed in #9741

@mitsudome-r mitsudome-r changed the title feat(diagnostic_graph_utils)!: tier4_debug_msgs changed to autoware_internal_debug_msgs in diagnostic_graph_utils feat(diagnostic_graph_utils): remove unnecessary dependency in diagnostic_graph_utils Jan 22, 2025
@mitsudome-r
Copy link
Member

I have changed the title to just change the package xml to remove the dependency to unused tier4_debug_msgs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:system System design and integration. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)
Projects
Status: To Triage
Development

Successfully merging this pull request may close these issues.

2 participants