Skip to content

Conversation

@SteveMacenski
Copy link
Member

@SteveMacenski SteveMacenski commented Nov 20, 2025

@Sushant-Chavan @nourazzii can one of you review that this successfully addresses your concern?

Signed-off-by: SteveMacenski <stevenmacenski@gmail.com>
@codecov
Copy link

codecov bot commented Nov 20, 2025

Codecov Report

❌ Patch coverage is 46.66667% with 8 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
nav2_controller/src/controller_server.cpp 46.66% 8 Missing ⚠️
Files with missing lines Coverage Δ
nav2_controller/src/controller_server.cpp 83.37% <46.66%> (-1.28%) ⬇️

... and 12 files with indirect coverage changes

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

@nourazzii
Copy link

LGTM :)

@SteveMacenski SteveMacenski merged commit d93fead into main Nov 25, 2025
14 of 15 checks passed
@mergify mergify bot deleted the none_progress_checker branch November 25, 2025 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants