Skip to content

chore(common, map): remove unused lanelet2_extension header#903

Merged
sasakisasaki merged 2 commits intoautowarefoundation:mainfrom
sarun-hub:chore/map-common/remove-unused-lanelet2-extension
Mar 11, 2026
Merged

chore(common, map): remove unused lanelet2_extension header#903
sasakisasaki merged 2 commits intoautowarefoundation:mainfrom
sarun-hub:chore/map-common/remove-unused-lanelet2-extension

Conversation

@sarun-hub
Copy link
Copy Markdown
Contributor

Description

Several PRs already ported functions from autoware_lanelet2_extension to autoware_lanelet2_utils.
During the porting process, some header and depends are not removed after replacing usage.
This PR cleans it.

Target components:

  • common
  • map

Related links

#894
Parent Issue:

  • Link

How was this PR tested?

Pass CI and Evaluator (https://evaluation.tier4.jp/evaluation/reports/bff44f2c-cf63-53f7-ba73-4fc0fbce4649?project_id=autoware_dev)

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

Signed-off-by: Sarun Mukdapitak <sarun.mukda@gmail.com>
Signed-off-by: Sarun Mukdapitak <sarun.mukda@gmail.com>
@sarun-hub sarun-hub self-assigned this Mar 11, 2026
@sarun-hub sarun-hub added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Mar 11, 2026
@github-actions
Copy link
Copy Markdown

Thank you for contributing to the Autoware project!

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

Please ensure:

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.30%. Comparing base (fecc9c3) to head (3e4b7b0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #903   +/-   ##
=======================================
  Coverage   50.30%   50.30%           
=======================================
  Files         367      367           
  Lines       22982    22982           
  Branches    10191    10191           
=======================================
  Hits        11560    11560           
  Misses      10307    10307           
  Partials     1115     1115           
Flag Coverage Δ *Carryforward flag
daily-humble 50.56% <ø> (ø) Carriedforward from fecc9c3
daily-jazzy 50.04% <ø> (ø) Carriedforward from fecc9c3
differential-humble 38.37% <ø> (?)
differential-jazzy 38.36% <ø> (?)
total 50.38% <ø> (ø) Carriedforward from fecc9c3
total-humble 50.56% <ø> (ø) Carriedforward from fecc9c3
total-jazzy 50.04% <ø> (ø) Carriedforward from fecc9c3

*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
Copy Markdown
Contributor

@sasakisasaki sasakisasaki left a comment

Choose a reason for hiding this comment

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

Nice refactoring! I agree with merging this PR 👍

@sasakisasaki sasakisasaki merged commit 771964d into autowarefoundation:main Mar 11, 2026
50 of 55 checks passed
NorahXiong pushed a commit to NorahXiong/autoware_core that referenced this pull request Mar 12, 2026
…foundation#903)

* remove unused lanelet2_extension in map component

Signed-off-by: Sarun Mukdapitak <sarun.mukda@gmail.com>

* remove unused lanelet2_extension in common component

Signed-off-by: Sarun Mukdapitak <sarun.mukda@gmail.com>

---------

Signed-off-by: Sarun Mukdapitak <sarun.mukda@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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