Skip to content

fix(autoware_behavior_velocity_planner): fix bugprone-narrowing-conversions warnings#939

Merged
mitsudome-r merged 1 commit intoautowarefoundation:mainfrom
NorahXiong:fix_bugprone-narrowing-conversions/autoware_behavior_velocity_planner
Mar 18, 2026
Merged

fix(autoware_behavior_velocity_planner): fix bugprone-narrowing-conversions warnings#939
mitsudome-r merged 1 commit intoautowarefoundation:mainfrom
NorahXiong:fix_bugprone-narrowing-conversions/autoware_behavior_velocity_planner

Conversation

@NorahXiong
Copy link
Contributor

Description

Step 2 of #774 (comment): Remove the exception from the .clang-tidy-ci list.

Related links

Parent Issue:
#774

How was this PR tested?

No clang-tidy error appears in CI.

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

@github-actions
Copy link

github-actions bot commented Mar 13, 2026

Thank you for contributing to the Autoware project!

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

Please ensure:

@NorahXiong NorahXiong added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Mar 13, 2026
@NorahXiong NorahXiong requested review from mitsudome-r and xmfcx March 13, 2026 05:26
@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 50.41%. Comparing base (1acc034) to head (b026212).
⚠️ Report is 14 commits behind head on main.

Files with missing lines Patch % Lines
...ehavior_velocity_planner/src/experimental/node.cpp 0.00% 1 Missing ⚠️
...er/autoware_behavior_velocity_planner/src/node.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #939      +/-   ##
==========================================
- Coverage   50.44%   50.41%   -0.03%     
==========================================
  Files         367      369       +2     
  Lines       23002    23012      +10     
  Branches    10215    10215              
==========================================
- Hits        11603    11602       -1     
- Misses      10280    10291      +11     
  Partials     1119     1119              
Flag Coverage Δ *Carryforward flag
daily-humble 50.71% <ø> (+<0.01%) ⬆️ Carriedforward from 1acc034
daily-jazzy 50.19% <ø> (+<0.01%) ⬆️ Carriedforward from 1acc034
differential-humble 8.70% <0.00%> (?)
total 50.38% <ø> (+<0.01%) ⬆️ Carriedforward from 1acc034
total-humble 50.70% <ø> (+<0.01%) ⬆️ Carriedforward from 1acc034
total-jazzy 50.19% <ø> (+<0.01%) ⬆️ Carriedforward from 1acc034

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

@mitsudome-r mitsudome-r force-pushed the fix_bugprone-narrowing-conversions/autoware_behavior_velocity_planner branch from 0275659 to b026212 Compare March 18, 2026 07:04
@mitsudome-r mitsudome-r enabled auto-merge (squash) March 18, 2026 07:04
@mitsudome-r mitsudome-r merged commit 7e01918 into autowarefoundation:main Mar 18, 2026
23 of 25 checks passed
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