Skip to content

Conversation

saikishor
Copy link
Member

I've just realized that somehow this part got changed in the merged #2449.

Sorry about that.

Copy link

codecov bot commented Sep 8, 2025

Codecov Report

❌ Patch coverage is 0% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.28%. Comparing base (ddb7fa9) to head (c3991f5).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
controller_manager/src/controller_manager.cpp 0.00% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2522      +/-   ##
==========================================
- Coverage   89.28%   89.28%   -0.01%     
==========================================
  Files         145      145              
  Lines       16479    16484       +5     
  Branches     1395     1395              
==========================================
+ Hits        14714    14717       +3     
- Misses       1225     1228       +3     
+ Partials      540      539       -1     
Flag Coverage Δ
unittests 89.28% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
controller_manager/src/controller_manager.cpp 75.18% <0.00%> (-0.10%) ⬇️

... and 1 file with indirect coverage changes

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

@christophfroehlich christophfroehlich merged commit 31c4bb1 into master Sep 8, 2025
28 checks passed
@christophfroehlich christophfroehlich deleted the readd/ResourceManagerParams branch September 8, 2025 10:40
@christophfroehlich
Copy link
Contributor

@Mergifyio backport mergify/bp/jazzy/pr-2449

Copy link
Contributor

mergify bot commented Sep 8, 2025

backport mergify/bp/jazzy/pr-2449

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Sep 8, 2025
christophfroehlich pushed a commit that referenced this pull request Sep 8, 2025
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.

2 participants