Skip to content

GPL custom validator: Use tl_expected from libexpected-dev instead#2212

Merged
christophfroehlich merged 6 commits intomasterfrom
tl-expected
Mar 25, 2026
Merged

GPL custom validator: Use tl_expected from libexpected-dev instead#2212
christophfroehlich merged 6 commits intomasterfrom
tl-expected

Conversation

@christophfroehlich
Copy link
Copy Markdown
Member

@christophfroehlich christophfroehlich marked this pull request as ready for review March 23, 2026 13:33
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 23, 2026

Codecov Report

❌ Patch coverage is 78.26087% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.78%. Comparing base (a014322) to head (db0f83a).
⚠️ Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
..._trajectory_controller/validate_jtc_parameters.hpp 78.26% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2212      +/-   ##
==========================================
- Coverage   84.80%   84.78%   -0.02%     
==========================================
  Files         153      153              
  Lines       15244    15234      -10     
  Branches     1321     1321              
==========================================
- Hits        12927    12916      -11     
- Misses       1835     1836       +1     
  Partials      482      482              
Flag Coverage Δ
unittests 84.78% <78.26%> (-0.02%) ⬇️

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

Files with missing lines Coverage Δ
..._trajectory_controller/validate_jtc_parameters.hpp 79.16% <78.26%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

saikishor
saikishor previously approved these changes Mar 23, 2026
@christophfroehlich christophfroehlich merged commit 813ed25 into master Mar 25, 2026
17 checks passed
@christophfroehlich christophfroehlich deleted the tl-expected branch March 25, 2026 19:20
@christophfroehlich christophfroehlich added backport-humble Triggers PR backport to ROS 2 humble. backport-jazzy Triggers PR backport to ROS 2 jazzy. backport-kilted Triggers PR backport to ROS 2 kilted. labels Mar 26, 2026
mergify bot pushed a commit that referenced this pull request Mar 26, 2026
(cherry picked from commit 813ed25)

# Conflicts:
#	joint_trajectory_controller/CMakeLists.txt
#	joint_trajectory_controller/include/joint_trajectory_controller/validate_jtc_parameters.hpp
mergify bot pushed a commit that referenced this pull request Mar 26, 2026
mergify bot pushed a commit that referenced this pull request Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-humble Triggers PR backport to ROS 2 humble. backport-jazzy Triggers PR backport to ROS 2 jazzy. backport-kilted Triggers PR backport to ROS 2 kilted.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants