Skip to content

Fix joint limits namespace wording in docstrings (backport #3445)#3447

Merged
christophfroehlich merged 1 commit into
jazzyfrom
mergify/bp/jazzy/pr-3445
Jul 6, 2026
Merged

Fix joint limits namespace wording in docstrings (backport #3445)#3447
christophfroehlich merged 1 commit into
jazzyfrom
mergify/bp/jazzy/pr-3445

Conversation

@mergify

@mergify mergify Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Update the return-value documentation for two get_joint_limits overloads.
  • Replace the slash-based parameter wording with the dotted
    joint_limits.joint_name namespace format.
  • Clarify that the general joint-limits overload succeeds when at least one
    recognized parameter exists under the namespace.

Fixes #3419

Testing

@codecov

codecov Bot commented Jul 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.42%. Comparing base (0194fdd) to head (47f175e).

Additional details and impacted files
@@            Coverage Diff             @@
##            jazzy    #3447      +/-   ##
==========================================
- Coverage   89.44%   89.42%   -0.03%     
==========================================
  Files         161      161              
  Lines       20567    20568       +1     
  Branches     1630     1630              
==========================================
- Hits        18397    18392       -5     
- Misses       1500     1506       +6     
  Partials      670      670              
Flag Coverage Δ
unittests 89.42% <ø> (-0.03%) ⬇️

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

Files with missing lines Coverage Δ
...its/include/joint_limits/joint_limits_rosparam.hpp 59.42% <ø> (ø)

... and 2 files 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 1a27987 into jazzy Jul 6, 2026
15 checks passed
@christophfroehlich christophfroehlich deleted the mergify/bp/jazzy/pr-3445 branch July 6, 2026 14:38
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