Skip to content

chore(autoware_localization_util): minor refactoring for header wrap#874

Merged
sasakisasaki merged 4 commits intoautowarefoundation:mainfrom
sasakisasaki:chore-minor-refactoring-for-header-wrap-in-autoware-localization-util
Mar 3, 2026
Merged

chore(autoware_localization_util): minor refactoring for header wrap#874
sasakisasaki merged 4 commits intoautowarefoundation:mainfrom
sasakisasaki:chore-minor-refactoring-for-header-wrap-in-autoware-localization-util

Conversation

@sasakisasaki
Copy link
Contributor

Description

Wrap the public headers with <> not "" for consistency

How was this PR tested?

  • (Ongoing) We see if the CI build/tests succeed

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
@github-actions
Copy link

github-actions bot commented Feb 27, 2026

Thank you for contributing to the Autoware project!

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

Please ensure:

@sasakisasaki sasakisasaki added component:localization Vehicle's position determination in its environment. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) labels Feb 27, 2026
@codecov
Copy link

codecov bot commented Feb 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.31%. Comparing base (d091381) to head (8e88caa).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #874      +/-   ##
==========================================
- Coverage   50.35%   50.31%   -0.05%     
==========================================
  Files         358      356       -2     
  Lines       22780    22727      -53     
  Branches    10120    10095      -25     
==========================================
- Hits        11472    11434      -38     
+ Misses      10203    10193      -10     
+ Partials     1105     1100       -5     
Flag Coverage Δ *Carryforward flag
daily-humble 50.69% <ø> (-0.04%) ⬇️ Carriedforward from af43bf1
daily-jazzy 50.18% <ø> (-0.04%) ⬇️ Carriedforward from af43bf1
differential-humble 60.56% <ø> (?)
total 50.38% <ø> (ø) Carriedforward from af43bf1
total-humble 50.69% <ø> (-0.04%) ⬇️ Carriedforward from af43bf1
total-jazzy 50.18% <ø> (-0.04%) ⬇️ Carriedforward from af43bf1

*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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@Motsu-san Motsu-san left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMeow

@sasakisasaki sasakisasaki requested a review from interimadd March 2, 2026 06:36
@sasakisasaki
Copy link
Contributor Author

@Motsu-san Thank you for your review! Is this your cat? 😄

Copy link
Contributor

@interimadd interimadd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sasakisasaki sasakisasaki enabled auto-merge (squash) March 3, 2026 11:35
@sasakisasaki sasakisasaki merged commit 273b8ac into autowarefoundation:main Mar 3, 2026
35 of 43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:localization Vehicle's position determination in its environment. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants